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

chore: allow release-plz to open changelog PRs #31

Merged
merged 1 commit into from
Nov 20, 2023
Merged

Conversation

olizilla
Copy link
Contributor

  • grant permissions to open PRs and add changelogs, needed for release-please to do the thing
  • run check and test in parallel. no need to have tests wait for lints.

hoping this fixes the error seen here where release please is unable to open a PR
https://github.com/web3-storage/files-from-path/actions/runs/6932934315/job/18858238223

see: https://github.com/google-github-actions/release-please-action/issues/709

License: MIT

- grant permissions to open PRs and add changelogs, needed for release-please to do the thing
- run check and test in parallel. no need to have tests wait for lints.

see: https://github.com/google-github-actions/release-please-action/issues/709

License: MIT
Signed-off-by: Oli Evans <[email protected]>
@@ -17,11 +17,9 @@ jobs:
node-version: 18
cache: 'npm'
- run: npm ci
- run: npm run build
Copy link
Contributor Author

Choose a reason for hiding this comment

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

build is redundant here, now we call it from prepare it happens as part of npm ci

- run: npm run lint

test:
needs: check
Copy link
Contributor Author

Choose a reason for hiding this comment

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

we can run check and test in parallel.

Comment on lines +45 to +46
contents: write
pull-requests: write
Copy link
Contributor Author

Choose a reason for hiding this comment

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

@olizilla olizilla merged commit 350c916 into main Nov 20, 2023
6 checks passed
@olizilla olizilla deleted the fix-release-plz branch November 20, 2023 16:48
olizilla pushed a commit that referenced this pull request Nov 20, 2023
🤖 I have created a release *beep* *boop*
---


##
[1.0.2](v1.0.1...v1.0.2)
(2023-11-20)


### Bug Fixes

* handle paths as string param
([#29](#29))
([fa7dcf0](fa7dcf0))
* #28
([fa7dcf0](fa7dcf0))


### Other Changes

* allow release-plz to open changelog PRs
([#31](#31))
([350c916](350c916))
* release from ci and add `--provenance` for verifiable builds!
([#30](#30))
([7ae47f4](7ae47f4))

---
This PR was generated with [Release
Please](https://github.com/googleapis/release-please). See
[documentation](https://github.com/googleapis/release-please#release-please).

Co-authored-by: github-actions[bot] <41898282+github-actions[bot]@users.noreply.github.com>
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