Skip to content

Extract Sentry initialization code into dedicated function #3920

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

Merged
merged 2 commits into from
Sep 19, 2021

Conversation

Turbo87
Copy link
Member

@Turbo87 Turbo87 commented Sep 12, 2021

This will allow us to more easily enable Sentry error reporting for the other binaries too.

@Turbo87 Turbo87 added C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear A-backend ⚙️ labels Sep 12, 2021
Copy link
Member

@JohnTitor JohnTitor left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@JohnTitor
Copy link
Member

@bors r+

@bors
Copy link
Contributor

bors commented Sep 19, 2021

📌 Commit b571353 has been approved by JohnTitor

@bors
Copy link
Contributor

bors commented Sep 19, 2021

⌛ Testing commit b571353 with merge 5f13e07...

@bors
Copy link
Contributor

bors commented Sep 19, 2021

☀️ Test successful - checks-actions
Approved by: JohnTitor
Pushing 5f13e07 to master...

@bors bors merged commit 5f13e07 into rust-lang:master Sep 19, 2021
@Turbo87 Turbo87 deleted the sentry branch September 19, 2021 19:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
A-backend ⚙️ C-internal 🔧 Category: Nonessential work that would make the codebase more consistent or clear
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants