Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

fix pnpm-setup workflow on release.yml #2567

Merged
merged 1 commit into from
Aug 20, 2024
Merged

fix pnpm-setup workflow on release.yml #2567

merged 1 commit into from
Aug 20, 2024

Conversation

anonrig
Copy link
Member

@anonrig anonrig commented Aug 20, 2024

It seems version field is not required whenever there is a packageManager field in package.json. Shout out to @npaun for finding the bug. Ref: https://github.com/pnpm/action-setup?tab=readme-ov-file#version

@anonrig anonrig requested review from jasnell and npaun August 20, 2024 19:01
@anonrig anonrig requested review from a team as code owners August 20, 2024 19:01
@anonrig anonrig merged commit 066a1a2 into main Aug 20, 2024
10 checks passed
@anonrig anonrig deleted the yagiz/fix-release-yml branch August 20, 2024 19:48
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants