ci: provide npm publish from PR pipeline
@pradel , This might be useful for proposals testing
@pradel , What's your vision here?
@antongolub sounds good, I will review the prs once I have a bit of time, these days are quite busy (in about 1/2 weeks)
Should be rebased after #77
@antongolub is this ready to be merged?
@pradel I want to believe. The only way to verify the pipeline is to run.
@antongolub can we move the pipeline logic to the release-please.yml file? The repo is now publishing to npm using OIDC instead of an NPM_TOKEN for improved security and npm only allows 1 file per repo to access the OIDC credentials, so all the logic to publish needs to be in the same file unfortunately
@pradel,
There are several options:
- Create additional trusted publisher entry for
release-from-pr.yml - Trigger subflows from root
release.ymland pass down the token torelease-pleaseandrelease-from-pr. - Create a single complex godlike publish file
Create additional trusted publisher entry for release-from-pr.yml
Not an option on npm unfortunately, there can be only 1 file for this
Trigger subflows from root release.yml and pass down the token to release-please and release-from-pr.
I don't think this would work as they check on which action the token is used
Create a single complex godlike publish file
I don't like this approach but it looks like it's the only option..