Repository navigation
docs: Require issue discussion before PRs - #6919
Merged
Merged
Conversation
Co-Authored-By: Chris Aigner <christoph.aigner@sentry.io>
Contributor
Semver Impact of This PR⚪ None (no version bump detected) 📋 Changelog PreviewThis is how your changes will appear in the changelog. This PR will not appear in the changelog. 🤖 This preview updates automatically when you update the PR. |
Contributor
Co-Authored-By: Chris Aigner <christoph.aigner@sentry.io>
Co-Authored-By: Chris Aigner <christoph.aigner@sentry.io>
christophaigner
marked this pull request as ready for review
October 22, 2026 08:49
christophaigner
requested review from
alwx,
antonis and
lucas-zimerman
as code owners
October 22, 2026 08:49
antonis
approved these changes
Oct 22, 2026
antonis
left a comment
antonis
left a comment
Contributor
There was a problem hiding this comment.
LGTM. Added a small suggestion regarding the logo visibility in dark mode
antonis
approved these changes
Oct 22, 2026
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
📢 Type of change
📜 Description
Asks external contributors to discuss a change with a maintainer in a GitHub issue before opening a PR. PRs without that prior discussion may be closed.
Changes to
CONTRIBUTING.md:discord.com/invite/sentry).Submitting Changessection with:Before You Start: find or create an issue, agree on the approach with a maintainer, check the assigneeMaking Your Contribution: tests are required for non-trivial PRs, don't bump version numbers, open a draft PR that links the issue, follow the commit message format, keep PRs focusedAI Use, moved up from the end of the file. It now also says contributors must understand the changes they propose, not just review and test them.💡 Motivation and Context
Aligns with getsentry/sentry-cocoa#9396 and getsentry/sentry-java#6288 so the mobile SDKs share the same contribution policy, with only the repo name changed. The wording says we may close PRs rather than promising automatic closing, because the
validate-prworkflow only posts an advisory comment.💚 How did you test it?
Docs-only change; reviewed the Markdown structure.
#skip-changelog
via Chris Aigner.
--
View Junior Session [Sentry]