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

Upgrading minimum node engine to 16 #2023

Merged

Conversation

jasonbarry
Copy link
Contributor

@jasonbarry jasonbarry commented Mar 27, 2022

Addresses #2022

Summary
This PR upgrades the minimum node engine version from 12 to 16, as node version 12 is approaching end-of-life on 04-30-2022. Specifying the minimum version of 16 will allow us to use dependencies that require a minimum version of 16.

Checklist

  • The basics
    • I tested these changes manually in my local or dev environment
  • Tests
    • Added or updated
    • N/A
  • Event Tracking
    • I added event tracking and followed the event tracking guidelines
    • N/A
  • Error Reporting
    • I reported errors and followed the error reporting guidelines
    • N/A

Screenshots
N/A

@jasonbarry jasonbarry requested a review from a team as a code owner March 27, 2022 20:28
@jasonbarry jasonbarry requested review from daniel-white and removed request for a team March 27, 2022 20:28
@daniel-white
Copy link
Member

hey @jasonbarry thanks for this! ill take a look, but in the mean time, can you bump this to node16 - that gives us more time and apple silicon support.

@jasonbarry jasonbarry changed the title Upgrading minimum node engine to 14 Upgrading minimum node engine to 16 Mar 28, 2022
@jasonbarry
Copy link
Contributor Author

@daniel-white just updated references to point to v16.

@daniel-white
Copy link
Member

hey @jasonbarry - sorry for the delay - can you rebase to pickup the actions changes?

@jasonbarry
Copy link
Contributor Author

@daniel-white sure I can do that, should I also change it so that it's a range instead of a min version? e.g. >=12 could avoid needing to make a potentially breaking change.

@daniel-white
Copy link
Member

@jasonbarry lets keep it as a range that 14 or 16 is the minimum. i think were good for potential breaking changes as we are planning to upgrade a bunch of dependencies that may introduce breaking changes as well.

@jasonbarry
Copy link
Contributor Author

@daniel-white oh I was thinking of a different PR, this one is already set to >=16. Just merged in changes from latest master.

@daniel-white daniel-white enabled auto-merge (squash) April 13, 2022 17:16
@daniel-white daniel-white merged commit d1e0817 into stoplightio:master Apr 13, 2022
@deefactorial
Copy link

deefactorial commented Jun 29, 2022

I'm not happy with this change, it's a breaking change that didn't change the major version. this even breaks previous versions as it tries to install the package @stoplight/prism-core at 4.10.0 if you try to install the package @stoplight/prism-cli at 4.8.0. I have an issue with upgrading to node 16 with another package so my only solution is to remove this package until I can find a way to upgrade to node 16. this breaking change could have been managed differently.

@daniel-white
Copy link
Member

hey @deefactorial - i'm sorry that this was a breaking change for you. we discussed this internally and #2085 was written as a result. to summarize some of outcomes:

  1. short term, we'll mark @stoplight/prism-cli to be explicit with its versions it depends on to avoid that another prism package would be upgraded
  2. we'll be discussing and documenting our decisions on versions we support and when breaking changes will be declared (or not)

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

Successfully merging this pull request may close these issues.

None yet

3 participants