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: run on node 20 #5

Merged
merged 3 commits into from
Feb 11, 2024
Merged

chore: run on node 20 #5

merged 3 commits into from
Feb 11, 2024

Conversation

DimitriPapadopoulos
Copy link
Collaborator

@DimitriPapadopoulos DimitriPapadopoulos commented Feb 10, 2024

Upgrade to node20 before bumping the version to v2 to fix #3.

@DimitriPapadopoulos
Copy link
Collaborator Author

Because pre-commit has been enabled for the whole organisation, we need to add a .pre-commit-config.yaml. Even an empty one should do.

ci_config

Avoid this CI error message:
	=====> .pre-commit-config.yaml is not a file
@DimitriPapadopoulos
Copy link
Collaborator Author

@peternewman Could you approve the workflow awaiting approval?

@peternewman
Copy link
Collaborator

Done and hopefully you'll now have the necessary permissions yourself too!

@DimitriPapadopoulos DimitriPapadopoulos merged commit a370d3c into codespell-project:master Feb 11, 2024
2 checks passed
@DimitriPapadopoulos DimitriPapadopoulos added the dependencies Pull requests that update a dependency file label Feb 11, 2024
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
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Deprecation warnings thrown in CI
2 participants