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

chore(deps): upgrade to node v20 #536

Merged
merged 1 commit into from
Jun 3, 2024

Conversation

agilgur5
Copy link

@agilgur5 agilgur5 commented Feb 12, 2024

Supersedes / Closes #432

Motivation

Modifications

  • update .nvmrc to Node v20

  • update all actions/setup-node GH Actions to use the .nvmrc for consistency EDIT: that ended up being completed as part of refactor(deps): migrate from archived tslint to eslint #509

  • due to an old version of Storybook / Webpack, we have to run all Storybook scripts with NODE_OPTIONS='--openssl-legacy-provider'

    • this must be used if we use newer versions of node without upgrading Storybook + Webpack
    • note that this is an insecure run-time configuration (which is why it was deprecated and is legacy), but since it is only used in development, it's not a significant vulnerability

Verification

Install works, builds pass, tests pass

Future Work

  1. Storybook upgrade, maybe never.
  2. Webpack upgrade, maybe doable. Then can remove --openssl-legacy-provider
  3. Have dependabot stop updating deps with breaking changes... 😐

@agilgur5 agilgur5 added type/dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Feb 12, 2024
- node v14 is EoL and has been [since April 2023](https://github.com/nodejs/Release/blob/68840fae8fdaa59bf9431661238dbb86b4390829/README.md?plain=1#L7)
- node v20 is the [current LTS version](https://github.com/nodejs/Release/blob/9cf9c6ed8178e88c74dfa1921a793701e1d162c4/README.md?plain=1#L8)

- due to an old version of Storybook / Webpack, we have to run all Storybook scripts with `NODE_OPTIONS='--openssl-legacy-provider'`
  - this must be used if we use newer versions of node without upgrading Storybook + Webpack
    - unfortunately, upgrading Storybook is significantly easier said than done, especially since there are 3 different Storybook builds in this repo (v1, v2, and `antd`)
      - so put that off for now (or ever? since this repo is not maintained much anymore)
  - note that this is an insecure run-time configuration (which is why it was deprecated and is legacy), but since it is only used in development, it's not a significant vulnerability

Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
@agilgur5 agilgur5 force-pushed the chore-deps-upgrade-node-20 branch from 09ed165 to 669bef4 Compare February 14, 2024 17:43
@agilgur5
Copy link
Author

Fixed merge conflicts

Copy link
Member

@csantanapr csantanapr left a comment

Choose a reason for hiding this comment

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

LGTM

.nvmrc Show resolved Hide resolved

This comment was marked as resolved.

@github-actions github-actions bot added the problem/stale This has not had a response in some time label Apr 23, 2024
@agilgur5 agilgur5 removed the problem/stale This has not had a response in some time label Apr 23, 2024
@agilgur5 agilgur5 merged commit af33af0 into argoproj:master Jun 3, 2024
5 checks passed
@agilgur5 agilgur5 deleted the chore-deps-upgrade-node-20 branch June 3, 2024 16:55
Adrien-D pushed a commit to Adrien-D/argo-ui that referenced this pull request Jun 6, 2024
- node v14 is EoL and has been [since April 2023](https://github.com/nodejs/Release/blob/68840fae8fdaa59bf9431661238dbb86b4390829/README.md?plain=1#L7)
- node v20 is the [current LTS version](https://github.com/nodejs/Release/blob/9cf9c6ed8178e88c74dfa1921a793701e1d162c4/README.md?plain=1#L8)

- due to an old version of Storybook / Webpack, we have to run all Storybook scripts with `NODE_OPTIONS='--openssl-legacy-provider'`
  - this must be used if we use newer versions of node without upgrading Storybook + Webpack
    - unfortunately, upgrading Storybook is significantly easier said than done, especially since there are 3 different Storybook builds in this repo (v1, v2, and `antd`)
      - so put that off for now (or ever? since this repo is not maintained much anymore)
  - note that this is an insecure run-time configuration (which is why it was deprecated and is legacy), but since it is only used in development, it's not a significant vulnerability

Signed-off-by: Anton Gilgur <agilgur5@gmail.com>
Signed-off-by: Adrien Delannoy <a.delannoyfr@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
javascript Pull requests that update Javascript code type/dependencies Pull requests that update a dependency file
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants