Setting up pkg.pr.new #567
Conversation
|
Currently failing with this as I don't have permissions to install it on this repo:
|
There was a problem hiding this comment.
Thank you for working on this! Just wanted to clarify why we need this change to the release process:
- we have a release process with versioning now, and scheduled releases every week on Thursday
- running builds on every commit to every branch is wasteful - unless I misunderstand the config, sorry
|
The point is, for PRs like #717 (and #687 which introduced the bug), you can properly test that PR in projects that depend on the SDK. For example, for https://github.com/modelcontextprotocol/use-mcp, the first I saw of the regression was when As for wastefulness, we already run builds on every commit/branch. This just saves the resulting package somewhere useful. |
I think more importantly, we do not auto-publish this SDK, we need an approval for publish. |
This publishes NPM-compatible URLs for each commit on the repo:
This makes it possible for people (like myself!) to build against the SDK before it goes through a release, to validate that everything's working with the downstream packages.
It requires installing this GH app on the main repo: https://github.com/apps/pkg-pr-new
Tested this on the
geelenfork (see screenshots) but this code really needs to be added as a branch to the main repo, then someone can iterate until they're happy with the output. Instructions for customisation are here, but I find the default look pretty much fine.