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

ci(provenance): add missing options from setup doc #9433

Merged
merged 1 commit into from
May 29, 2024

Conversation

benelan
Copy link
Member

@benelan benelan commented May 28, 2024

Related Issue: #9429

Summary

Update the deployment GitHub Actions according to steps in the npm provenance doc, which I missed because I was looking at the 3rd party section below it.

The GitHub provenance doc doesn't mention permissions, so it might not be required.

We had originally hardcoded an older version of Ubuntu in #5942 due to errors, but it was resolved in Stencil v4.

Update the deployment GitHub Actions according to steps in the npm doc,
which I missed because I was looking at the 3rd party section below it.

The GitHub doc about provenance doesn't mention permissions, so it might
not be required.

We had originally hardcoded an older version of Ubuntu in #5942 due to
errors, but it was resolved in Stencil v4.

npm-doc: https://docs.npmjs.com/generating-provenance-statements#publishing-packages-with-provenance-via-github-actions
github-doc: https://docs.github.com/en/actions/publishing-packages/publishing-nodejs-packages#publishing-packages-to-the-npm-registry
@github-actions github-actions bot added the chore Issues with changes that don't modify src or test files. label May 28, 2024
@benelan benelan added the skip visual snapshots Pull requests that do not need visual regression testing. label May 28, 2024
Copy link
Member

@jcfranco jcfranco left a comment

Choose a reason for hiding this comment

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

🚀

@benelan benelan marked this pull request as ready for review May 29, 2024 03:38
@benelan benelan merged commit b390cd4 into main May 29, 2024
10 checks passed
@benelan benelan deleted the benelan/provenance-missing-settings branch May 29, 2024 03:39
benelan added a commit that referenced this pull request May 29, 2024
…hes--main

* origin/main:
  chore: release next
  ci: add repo url to package.json files for provenance (#9449)
  ci(provenance): add missing options from setup doc (#9433)
  chore: release hotfix (#9438)
  fix: defer floating-ui updating until component is connected and open (#9443)
  fix(combobox): fix error that occurs in dist-custom-elements (components) output when a click is emitted when the component is appended to the DOM (#9423)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
chore Issues with changes that don't modify src or test files. skip visual snapshots Pull requests that do not need visual regression testing.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants