watch bt releases on npm, update local bt version on update#2067
watch bt releases on npm, update local bt version on update#2067Cedric / ViaDézo1er (viadezo1er) wants to merge 1 commit into
Conversation
rum with a daily cron job or manually
Luca Forstner (lforst)
left a comment
There was a problem hiding this comment.
This is not what we want. Updates should be manual as a minor in bt is a minor in sdk, and a major in bt, is a major in sdk.
There was a problem hiding this comment.
This should not be a .cjs file I think. In general, let's follow https://github.com/getsentry/sentry-cli/blob/master/scripts/install.js or https://blog.sentry.io/publishing-binaries-on-npm/ to the tee. The logs are very intentional there and we should log similar things to ensure people are not confused when installing the package and things fail.
I also have an example repo on how this should look like: https://github.com/lforst/npm-binary-example
|
We'll stick with manual version update for now. |
|
Forgot we need to change the |
Run with a daily cron job or manually.
Regenerate
pnpm-lock.yamlonce thebtbinaries have been pushed tonpmto fix CI.Depends on braintrustdata/bt#208.