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

Use consistent node version for local, CI, and devcontainer setup #4424

Merged
merged 3 commits into from
Jan 9, 2025

Conversation

inverse
Copy link
Contributor

@inverse inverse commented Dec 24, 2024

@netmindz
Copy link
Collaborator

Do we need to be so specific right down to the patch version?

@inverse
Copy link
Contributor Author

inverse commented Dec 24, 2024

Can be modified to not include the patch version but would mean more consistent builds.

Leveraging tools such as renovate can automate bumping the patch version too.

Happy to drop the patch version if you feel it makes more sense for this project.

@netmindz
Copy link
Collaborator

Yeah if you can drop the patch version please. We don't have the resources to keep on top of patch updates

@@ -2,12 +2,7 @@

# [Choice] Python version: 3, 3.9, 3.8, 3.7, 3.6
ARG VARIANT="3"
FROM mcr.microsoft.com/vscode/devcontainers/python:0-${VARIANT}
Copy link
Contributor Author

Choose a reason for hiding this comment

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

This container has been deprecated since last year so migrated to the new version

@inverse inverse changed the title Use consistent node version Use consistent node version for local, CI, and devcontainer setup Dec 24, 2024
@inverse
Copy link
Contributor Author

inverse commented Jan 4, 2025

@netmindz any further feedback on this one?

@softhack007 softhack007 added dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code labels Jan 6, 2025
@netmindz netmindz merged commit 7208282 into Aircoookie:main Jan 9, 2025
19 checks passed
@inverse inverse deleted the consistent-node branch January 15, 2025 20:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
dependencies Pull requests that update a dependency file javascript Pull requests that update Javascript code
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants