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.

Sign up for GitHub

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

Jump to bottom

ci(release): Add support for release via getsentry/publish repo #3124

Merged
merged 1 commit into from
Dec 16, 2020
Merged

ci(release): Add support for release via getsentry/publish repo #3124

merged 1 commit into from
Dec 16, 2020

Conversation

BYK
Copy link
Member

@BYK BYK commented Dec 15, 2020

Same as getsentry/symbolicator#319.

#skip-changelog

@BYK BYK requested a review from kamilogorek as a code owner December 15, 2020 22:05
@BYK BYK requested a review from HazAT December 15, 2020 22:05
@BYK
Copy link
Member Author

BYK commented Dec 15, 2020

We need admin access for @getsentry/releases on this repo to make this work and also set up a needed secret @HazAT

@github-actions
Copy link
Contributor

size-limit report

Path Size
@sentry/browser - CDN Bundle (gzipped) 19.93 KB (-0.01% 🔽)
@sentry/browser - Webpack 20.76 KB (0%)
@sentry/react - Webpack 20.76 KB (0%)
@sentry/browser + @sentry/tracing - CDN Bundle (gzipped) 27.09 KB (-0.01% 🔽)

@jan-auer
Copy link
Member

Permissions added.

@jan-auer
Copy link
Member

@BYK workflow LGTM, you'll still have to add the tokens though.

@BYK BYK merged commit dfa97e4 into master Dec 16, 2020
@BYK BYK deleted the byk/ci/centralized-repo branch December 16, 2020 09:52
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@jan-auer jan-auer jan-auer approved these changes

@kamilogorek kamilogorek Awaiting requested review from kamilogorek

@HazAT HazAT Awaiting requested review from HazAT

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@BYK @jan-auer