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

Publish with Provenance #3426

Merged
merged 4 commits into from
Apr 21, 2024
Merged

Publish with Provenance #3426

merged 4 commits into from
Apr 21, 2024

Conversation

samchungy
Copy link
Contributor

@samchungy samchungy commented Apr 21, 2024

Zod Pull Request

IMPORTANT: Development of the next major version of Zod (v4) is currently ongoing. If your PR implements new functionality, it should target the v4 branch, NOT the master branch. (If it's a bugfix, the master branch is fine.)

Overview

https://docs.npmjs.com/generating-provenance-statements#example-github-actions-workflow

https://github.com/marketplace/actions/npm-publish#github-action

Gotta get you the ✅ on npm

Screenshot_20240421-182609

Not sure if contents needs to be write but I see in the CI that your pipeline pushes up to GitHub. Pull request read is for your change log step which appears to read pull requests.

  • Your code changes are well-documented.
  • You have tested your changes.
  • You have updated any relevant documentation.

Copy link

netlify bot commented Apr 21, 2024

Deploy Preview for guileless-rolypoly-866f8a ready!

Name Link
🔨 Latest commit b02888a
🔍 Latest deploy log https://app.netlify.com/sites/guileless-rolypoly-866f8a/deploys/66258b670066320008d630cc
😎 Deploy Preview https://deploy-preview-3426--guileless-rolypoly-866f8a.netlify.app
📱 Preview on mobile
Toggle QR Code...

QR Code

Use your smartphone camera to open QR code link.

To edit notification comments on pull requests, go to your Netlify site configuration.

@colinhacks
Copy link
Owner

Thanks! Tacked on some additional housekeeping - updating some actions to the latest versions.

@colinhacks colinhacks merged commit 6c6161f into colinhacks:master Apr 21, 2024
20 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