-
Notifications
You must be signed in to change notification settings - Fork 8
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
Pin older node version for prettier pre-commit hook #144
Conversation
for more information, see https://pre-commit.ci
A PR has been generated to the instance repo: You can check out the PR to preview your changes |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
TBH I'm not sure whether I want the version pinned in the template. Maybe something that should rather be in the docs?
How many people are affected by this? I saw the issue...
It affects everyone on Ubuntu 18.04, which I'm afraid is still rather common. |
Sadly you are correct. DW, I won't block this PR, but these kinds of fixes that at one point might get forgotten annoy me :) Thank you for looking into it! |
18.10 should work though. Not sure about other OS. The linked jupyterhub issue also mentions amazon linux. |
for more information, see https://pre-commit.ci
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thx
Fix #143