Skip to content

Upgrade node to latest LTS in templates#1896

Merged
Ilyesbdlala merged 1 commit intosecureCodeBox:mainfrom
o1oo11oo:fix/upgrade-node
Sep 12, 2023
Merged

Upgrade node to latest LTS in templates#1896
Ilyesbdlala merged 1 commit intosecureCodeBox:mainfrom
o1oo11oo:fix/upgrade-node

Conversation

@o1oo11oo
Copy link
Contributor

@o1oo11oo o1oo11oo commented Sep 5, 2023

I missed this node version when upgrading to latest LTS in #1853

Relates to #1852

Description

Checklist

  • Test your changes as thoroughly as possible before you commit them. Preferably, automate your test by unit/integration tests.
  • Make sure that all your commits are signed-off and that you are added to the Contributors file.
  • Make sure that all CI finish successfully.
  • Optional (but appreciated): Make sure that all commits are Verified.

@o1oo11oo o1oo11oo added dependencies Pull requests that update a dependency file maintenance ci Changes to the continuous integration setup labels Sep 5, 2023
@o1oo11oo o1oo11oo self-assigned this Sep 5, 2023
@Ilyesbdlala
Copy link
Member

As far as I can tell that variable is never used (and is not needed) right ? So we can probably just remove it.

@o1oo11oo
Copy link
Contributor Author

o1oo11oo commented Sep 5, 2023

Good point, I'm not sure, I didn't check the workflow that far. The other versions are also quite old (usually we use Go 1.19 I think).

@netlify
Copy link

netlify bot commented Sep 8, 2023

Deploy Preview for docs-securecodebox canceled.

Name Link
🔨 Latest commit 6161bc9
🔍 Latest deploy log https://app.netlify.com/sites/docs-securecodebox/deploys/650013e85db70e00083c628b

@o1oo11oo o1oo11oo changed the title Upgrade node to latest LTS in workflows Upgrade node to latest LTS in workflows and templates Sep 8, 2023
@o1oo11oo o1oo11oo changed the title Upgrade node to latest LTS in workflows and templates Upgrade node to latest LTS in templates Sep 11, 2023
@o1oo11oo
Copy link
Contributor Author

o1oo11oo commented Sep 11, 2023

I opened #1946 to remove the unused version variables, so this now only contains the change of the node version in the templates.

Missed this node version when upgrading to latest LTS in secureCodeBox#1853

Signed-off-by: Lukas Fischer <[email protected]>
@Ilyesbdlala Ilyesbdlala merged commit a18802a into secureCodeBox:main Sep 12, 2023
@o1oo11oo o1oo11oo deleted the fix/upgrade-node branch September 12, 2023 08:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ci Changes to the continuous integration setup dependencies Pull requests that update a dependency file maintenance

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

2 participants