-
-
Notifications
You must be signed in to change notification settings - Fork 31
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
Ref: Github Action for updating Sentry dependencies. #512
Conversation
scripts/update-javascript.sh
Outdated
|
||
tagPrefix='' | ||
repo="https://github.com/getsentry/sentry-javascript.git" | ||
packages=('@sentry/browser' '@sentry/core' '@sentry/integrations' '@sentry/react' '@sentry/vue' '@sentry/angular' '@sentry/angular-ivy' '@sentry/types' '@sentry/utils') |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I believe this won't work for @sentry/angular-ivy
, @sentry/angular
, @sentry/react
, and @sentry/vue
. As the update-package-json.sh
runs yarn upgrade
.
I tried to run yarn upgrade
and it added these as deps not upgrading the peers.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
nice catch!, sucks that yarn upgrade doesn't support peerDependencies...
I added another way of updating the peerDependencies by using the add command with the --peer flag instead of relying on the upgrade command
…into feat/auto-bump=dep
…ed current branch for testing
The PR will not pass for the time being because the update logic compares with the main branch and this branch isn't the main branch, but all the previous steps are passing |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
LGTM, thanks for adding this! 🚀
Close #510
#skip-changelog