Skip to content
This repository has been archived by the owner on Jan 14, 2023. It is now read-only.

Sentry #125

Merged
merged 10 commits into from
Nov 19, 2020
Merged

Sentry #125

merged 10 commits into from
Nov 19, 2020

Conversation

Yatekii
Copy link
Member

@Yatekii Yatekii commented Nov 6, 2020

I added OPTIONAL logging to sentry and would welcome opinions on it :)

@Yatekii Yatekii marked this pull request as ready for review November 9, 2020 21:10
@Yatekii Yatekii requested a review from Tiwalun November 9, 2020 21:10
Copy link
Contributor

@dbrgn dbrgn left a comment

Choose a reason for hiding this comment

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

I like it, as long as it's strictly opt-in 🙂

Where's that sentry server hosted, and who has access to it? Are the crash reports public?

README.md Outdated Show resolved Hide resolved
README.md Outdated Show resolved Hide resolved
@Yatekii
Copy link
Member Author

Yatekii commented Nov 14, 2020

Thanks for the proofread (do we write this as one word? oO)!
Yes it will be strictly opt in.

We have an OSS sponsored organization on sentry.io. Sadly it cannot be viewed without an account but anyone can join. So if you would like to see what's going on, I'll add you, no questions asked :)

Copy link
Member

@Tiwalun Tiwalun left a comment

Choose a reason for hiding this comment

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

bors r+

@bors
Copy link
Contributor

bors bot commented Nov 19, 2020

@bors bors bot merged commit d4d5b02 into master Nov 19, 2020
@bors bors bot deleted the sentry branch November 19, 2020 09:27
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants