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

Add .nvmrc file as specified for setup-node action #34

Merged
merged 1 commit into from
Mar 20, 2024

Conversation

Divs-B
Copy link
Contributor

@Divs-B Divs-B commented Mar 20, 2024

What does this change?

In this PR #32, we specified node-version-file in release.yml file to use .nvmrc file but we didn't add the file and hence got failed on NPM release. ref: https://github.com/guardian/content-entity/actions/runs/8360348474

How to test

Make a preview/production release to check NPM release as well this time

How can we measure success?

NPM release should go successfully.

In this PR #32, we specified node-version-file in release.yml file to use .nvmrc file but we didn't add the file and hence got failed on NPM release.
ref: https://github.com/guardian/content-entity/actions/runs/8360348474
@Divs-B Divs-B requested review from a team as code owners March 20, 2024 14:29
@Divs-B Divs-B requested a review from rtyley March 20, 2024 14:29
@gu-scala-library-release
Copy link
Contributor

@Divs-B has published a preview version of this PR with release workflow run #3, based on commit 464e7da:

3.0.1-PREVIEW.db-rtadd-nvmrc-file-for-release.2024-03-20T1430.464e7dae

Want to make another preview release?

Click 'Run workflow' in the GitHub UI, specifying the db-rt/add-nvmrc-file-for-release branch, or use the GitHub CLI command:

gh workflow run release.yml --ref db-rt/add-nvmrc-file-for-release

Want to make a full release after this PR is merged?

Click 'Run workflow' in the GitHub UI, leaving the branch as the default, or use the GitHub CLI command:

gh workflow run release.yml

Copy link
Member

@rtyley rtyley 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 this fixed it - preview release went fine!

image

@Divs-B Divs-B merged commit 222c33f into main Mar 20, 2024
9 checks passed
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.

2 participants