Skip to content

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)

3 participants