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

How to handle Node versioning? #14

Closed
georgemandis opened this issue Feb 2, 2020 · 4 comments
Closed

How to handle Node versioning? #14

georgemandis opened this issue Feb 2, 2020 · 4 comments

Comments

@georgemandis
Copy link
Owner

One of the solutions works in v11.4.0 but not other versions of Node. I blame my own lack of diligence.

Still, it raises an interesting question. I've got a GitHub action running the tests now. Should I pick a version of Node that all the solutions have to work under and stick with that? Should I explore a fancier fix where each solution can post something like it's own .nvmrc file and modify the tests to use the requests version of Node?

Decisions decisions...

@nderscore
Copy link
Contributor

nderscore commented Feb 14, 2020

Being on the bleeding edge of new language features will allow for more unique solutions, so it would be great if it could be kept up to date.

Dealing with managing node versions in order to run the tests sounds like a pain, though.

Since none of the solutions have been broken by a new version of node yet, I don't see why running CI tests on the latest version of node for the time being would be a problem. Cross that bridge if/when you get there? 😄

@georgemandis
Copy link
Owner Author

I agree bleeding edge features expand the possibilities.

The challenge here is that one of them only passes in Node 11.4.0 right now... and it’s one I contributed! I know why, but I’m sort of at a loss as to how to refactor it, mostly because it’s obnoxiously obfuscated and the only way I can think to fix it would give away some of the fun :)

@georgemandis
Copy link
Owner Author

It’s 023, just FYI.

@georgemandis
Copy link
Owner Author

Closing because #25 is more direct and actionable.

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

No branches or pull requests

2 participants