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

Add support for node version codenames and keywords #58

Open
wants to merge 1 commit into
base: master
from

Conversation

@JimiC
Copy link

@JimiC JimiC commented Sep 13, 2019

Resolves #26

This PR adds support for node codenames (argon, boron etc) and selected keywords (latest, current, lts).

Resolving the codenames and keywords is fully automated and ensures future codenames support.

In order to do so, I took the liberty to refactor the code and test cases for better optimization and maintainability.

I'm aware that code reviewing will be a nightmare because of the many changes. I suggest having a look at the new code itself instead, to have a better understanding of it.

In case the refactoring is not acceptable, I'll try to provide a PR with minimum changes, but the code will be ugly.

Refactored for optimization and maintainability
@JimiC
Copy link
Author

@JimiC JimiC commented Sep 13, 2019

Suggestion: Since the refactoring done in this PR interferes with #42, I could also incorporate those changes in this PR.

Edit: I also didn't update README until we finalize the PR.

@DanielRuf
Copy link

@DanielRuf DanielRuf commented Jan 23, 2020

latest would be great, as a solution you can already use '*'.

@ExE-Boss
Copy link

@ExE-Boss ExE-Boss commented Feb 20, 2020

This PR seems to have a lot of out of scope changes in it.

@JimiC
Copy link
Author

@JimiC JimiC commented Feb 20, 2020

Not going to update it if it's not going to be merged.

@wdanilo
Copy link

@wdanilo wdanilo commented Nov 2, 2020

Why is this not merged? It would be important to me to indicate that I want my tests to run on the latest LTS node version. Right now I need to update it everywhere by hand.

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

Successfully merging this pull request may close these issues.

4 participants
You can’t perform that action at this time.