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

Update Node version in Github Action workflow scripts #3351

Closed
wants to merge 23 commits into from

Conversation

lchoy
Copy link
Collaborator

@lchoy lchoy commented Nov 30, 2023

Modifies the version string in .nvmrc to remove the 'v' prefix to enable compatibility with GitHub Action workflow scripts. Update action workflow scripts to use the node-version-file.

@NickAkhmetov
Copy link
Collaborator

The actions should already be using the node version specified in the .nvmrc file; was there an issue of some sort?

@lchoy
Copy link
Collaborator Author

lchoy commented Dec 1, 2023

The version in the .nvmrc is not supported. See https://github.com/actions/setup-node#supported-version-syntax. As a result, the workflow produces this warning about overriding the default to 16:
portal-ui-workflow

I opened the PR (set to draft) to force the workflow to run. Goal is to use a consistent version of the action and node-version. Perhaps node-version: latest would be the desired option? Or perhaps a list with either latest or .nvmrc.

Also, am planning to remove the node-version-file: ".nvmrc" line before finalizing the PR as node-version takes precedence.

@NickAkhmetov
Copy link
Collaborator

NickAkhmetov commented Dec 1, 2023

The version in the .nvmrc is not supported.

The version in the .nvmrc is v20.7.0; maybe the v prefix is causing some issues? Could we try removing the v and seeing if GitHub actions properly detect it? It seems like that would still match the .nvmrc spec while matching the expected format indicated in the setup-node readme.

Regardless of whether the action uses the .nvmrc file or not (I still strongly prefer using the node-version-file approach to avoid having multiple sources of truth), we should avoid regressing to Node 18. I think "latest" would take us to 21.

NickAkhmetov and others added 21 commits December 13, 2023 09:19
* Add hooks to update workspaces

* Add store to control edit dialog

* Create basic workspace name dialog

* Reset form on open

* Handle disabled button tooltip

* Pass mouse events to tooltip button

* Fix update types

* Handle update errors

* Dedupe workspace stores

* Share workspace form fields

* Jump through typescript hoops

* Add step styling to modal

* Pull template select step into own component

* Pull out create templates hook for reuse

* Add edit templates dialog

* Match templates in current workspace details

* Add comment

* Disable templates already in workspace

* Do not select disabled templates when selecting all

* Add tooltip if template is already in workspace

* Create reusable edit workspace dialog

* Move workspace templates and datasets into swr hook

* Handle reset after submit successful

* Consolidate edit dialogs

* Add changelog

* Pass workspace datasets to new templates

* Fix TS errors related to template form types (#3353)

* Update context/app/static/js/shared-styles/cards/SelectableCard/SelectableCard.tsx

Co-authored-by: Nikolay Akhmetov <[email protected]>

* Use store instead of props

* Use padding prop in stack in step

* Use promise chain to avoid async submit callback

* Console error error

---------

Co-authored-by: John Conroy <[email protected]>
Co-authored-by: Nikolay Akhmetov <[email protected]>
* Add tutorials routes to flask

* Add tutorial route to js

* Add tutorials page

* Add workspace tutorial

* Use mui styles

* Add tutorial to workspaces page

* Add changelog

---------

Co-authored-by: John Conroy <[email protected]>
* NickAkhmetov/Fix organ counts

* remove unnecessary separate return
* Add fusion link and fix azimuth casing

* Add changelog

* Add tracking

* Fix callback

---------

Co-authored-by: John Conroy <[email protected]>
* Add utm source param to fusion link

* Add changelog

---------

Co-authored-by: John Conroy <[email protected]>
@lchoy lchoy closed this Jan 2, 2024
@lchoy lchoy deleted the lchoy/github-action-node-version-update branch January 2, 2024 21:34
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.

3 participants