-
Notifications
You must be signed in to change notification settings - Fork 2.4k
Super Upgrade PR #837
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
base: master
Are you sure you want to change the base?
Super Upgrade PR #837
Conversation
Use a version of ngrok which runs on recent versions of Windows and MacOS. Switch to using the npm `overrides` property, to avoid needing to mantain a separate fork of `zuul-ngrok`
Browser usage
Upgrade modules used in github actions to latest supported version
Upgrade action node versions
Manually install `esm` as part of the legacy test job, and ignore the lockfile
Upgrade npm dev dependencies
@jedidja no, I've not heard anything since I raised these PRs a year ago. I don't think this project is being actively maintained. |
Before people start pouring a lot of effort into a fork, please consider that there are more Mustache implementations for JavaScript:
(There is also Milk, but that one went out of maintenance almost immediately in 2011.) |
@jedidja I did publish my collective changes to npm last year, so I could include it in one of my projects: https://www.npmjs.com/package/mustache-dynamic-names But I don't really want to become the sole maintainer of a fork relied on by others. |
@jgonggrijp good luck with your library too! |
I've made a bunch of individual Pull Requests to upgrade parts of this repository.
However, merging them all will cause a bunch of conflicts. This pull request combines them all into a single PR with the conflicts already sorted.
This covers:
For the purposes of review, you may wish look at each of those individually to understand the changes.
(Fixes #832)