Skip to content

Remove guaranteed compatibility with versions to 10> and <15 #155

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

Merged
merged 5 commits into from
Jun 8, 2021

Conversation

bidoubiwa
Copy link
Contributor

@bidoubiwa bidoubiwa commented Jun 4, 2021

fixes #154

needs tests to be removed from mandatory tests in settings @curquiza

Since the builded package can be used (without guarantee) in other node versions, we should not make it mandatory to use the right node version. It is at the discretion of the user.

It is how most libraries handle node support. The use of engines in packages json limits the possibility to install the package in a specific node environment (see issue).
Whereas typically a package that is bundled and transpiled should be as compatible as possible with all js environments. But, the packages we use in development are not. This is why we add it in the CONTRIBUTING guide

@bidoubiwa bidoubiwa requested a review from curquiza June 4, 2021 13:52
@bidoubiwa bidoubiwa changed the title Limit node versions to 10> and <15 Remove guaranteed compatibility with versions to 10> and <15 Jun 4, 2021
Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors try

bors bot added a commit that referenced this pull request Jun 7, 2021
@bors
Copy link
Contributor

bors bot commented Jun 7, 2021

try

Build succeeded:

Co-authored-by: Clémentine Urquizar <[email protected]>
Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

bors try

bors bot added a commit that referenced this pull request Jun 7, 2021
@bors
Copy link
Contributor

bors bot commented Jun 7, 2021

try

Build succeeded:

@bidoubiwa
Copy link
Contributor Author

bors merge

@bors
Copy link
Contributor

bors bot commented Jun 8, 2021

Build succeeded:

@bors bors bot merged commit 480142a into main Jun 8, 2021
@bors bors bot deleted the limit_node_versions branch June 8, 2021 11:28
@curquiza
Copy link
Member

curquiza commented Jun 8, 2021

@bidoubiwa I put the breaking label to update the minor :)

@curquiza curquiza added the breaking-change The related changes are breaking for the users label Jun 8, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
breaking-change The related changes are breaking for the users
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Changes after the Node.js v10 EOL (2021-04-30)
2 participants