Skip to content

Use changesets for releases, use Node 20 #1154

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

Merged
merged 12 commits into from
May 12, 2025

Conversation

ericglau
Copy link
Member

@ericglau ericglau commented May 7, 2025

Uses changesets for versioning and publishing.
Updates build environment to Node.js v20.

Equivalent to OpenZeppelin/contracts-wizard#528

@ericglau ericglau requested review from CoveMB and tirumerla May 7, 2025 17:44
@ericglau ericglau changed the title Use changesets for releases Use changesets for releases, use Node 20 May 7, 2025
Copy link
Contributor

@tirumerla tirumerla left a comment

Choose a reason for hiding this comment

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

lgtm

title: Prepare Release
commit: Prepare Release
version: npm run version
commitMode: github-api
Copy link
Contributor

Choose a reason for hiding this comment

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

Is this specifically for signing commits or any other advantages over cli?

Copy link
Member Author

Choose a reason for hiding this comment

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

Yes, according to the Changesets changelog,

When used with "github-api" value all tags and commits will be attributed to the user whose GITHUB_TOKEN is used, and also signed using GitHub's internal GPG key.

commitMode: github-api
env:
GITHUB_TOKEN: ${{ secrets.GITHUB_TOKEN }}
NPM_TOKEN: ${{ secrets.NPM_TOKEN }}
Copy link
Contributor

Choose a reason for hiding this comment

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

For future reference making comment to enable config provenance when pushing

Copy link
Member Author

@ericglau ericglau May 12, 2025

Choose a reason for hiding this comment

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

Adding link to related comment for tracking: OpenZeppelin/contracts-wizard#528 (comment)

contents: write
pull-requests: write
runs-on: ubuntu-latest
environment: publish
Copy link
Contributor

Choose a reason for hiding this comment

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

❤️

@ericglau ericglau merged commit efb77e6 into OpenZeppelin:master May 12, 2025
14 checks passed
@ericglau ericglau deleted the changesets branch May 12, 2025 21:12
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants