Skip to content
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

update Node.js versions used in AppVeyor and Travis CI #568

Open
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

striezel
Copy link

@striezel striezel commented May 7, 2021

Node.js versions below 10.x are not maintained anymore, so it is probably best to use some newer, still maintained versions.
For release cyles of Node.js see the official site: https://nodejs.org/en/about/releases/.

Node.js versions below 10.x are not maintained anymore, so it
is probably best to use some newer, still maintained versions.
For release cyles of Node.js see the official site:
<https://nodejs.org/en/about/releases/>.
@striezel
Copy link
Author

@shama: I am aware that the AppVeyor build checks still fail, but they are failing for the current master commit (fc8458e), too. See https://ci.appveyor.com/project/gruntjs/grunt-contrib-watch/builds/38343248. However, Travis CI is fine again, so I consider this to be a step into the right direction.

@striezel
Copy link
Author

striezel commented Jul 3, 2021

To @shama or to whom it may concern: Is there anything wrong with this pull request? It's been almost two months now and no reaction so far. 🤔

@shama
Copy link
Member

shama commented Jul 3, 2021

Sorry I'm not an active maintainer on Grunt anymore.

@striezel
Copy link
Author

striezel commented Jul 4, 2021

@shama

Sorry I'm not an active maintainer on Grunt anymore.

That is a bit unexpected. :| Since you are the primary contributor to this repository and (as of now) you still see to be a member of the gruntjs organization here on GitHub, I assumed you are an active maintainer.

Could you please point me to a person who still is actively maintaining grunt and can review this pull request? Thanks.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants