Skip to content

Test removal of nvmrc#892

Open
aug24 wants to merge 1 commit intomainfrom
test-removal-of-nvmrc
Open

Test removal of nvmrc#892
aug24 wants to merge 1 commit intomainfrom
test-removal-of-nvmrc

Conversation

@aug24
Copy link
Copy Markdown

@aug24 aug24 commented Mar 18, 2026

What does this change?

We would like to standardise on the non-tool-specific tool versions definition file. This appears to be the direction of travel of many/most tools and it keeps our dotfile count down.

In this PR we test complete removal of .nvmrc in favour of .tool-versions, and also of specifying the node-version-file in ci.yaml, to confirm it supports the new location.

How has this change been tested?

How can we measure success?

Have we considered potential risks?

Images

Accessibility

@aug24 aug24 requested review from a team as code owners March 18, 2026 09:57
@aug24 aug24 force-pushed the test-removal-of-nvmrc branch from a227632 to 6d9717c Compare March 18, 2026 09:57
@aug24 aug24 added the maintenance Departmental tracking: maintenance work, not a fix or a feature label Mar 18, 2026
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 18, 2026

@aug24 aug24 force-pushed the test-removal-of-nvmrc branch from ef0aabb to cfd2592 Compare March 18, 2026 10:23
@aug24 aug24 force-pushed the test-removal-of-nvmrc branch from cfd2592 to 914d25d Compare March 18, 2026 10:24
Copy link
Copy Markdown
Member

@kelvin-chappell kelvin-chappell left a comment

Choose a reason for hiding this comment

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

Looks like it's working 👍
Maybe the PR description should have some rationale for doing this and what might go wrong. But seems unlikely anything will go wrong.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

maintenance Departmental tracking: maintenance work, not a fix or a feature

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants