Skip to content

docs: Require issue discussion before PRs - #6288

Draft
sentry-junior[bot] wants to merge 1 commit into
mainfrom
docs/require-issue-discussion
Draft

sentry-junior[bot] wants to merge 1 commit into
mainfrom
docs/require-issue-discussion

Conversation

@sentry-junior

@sentry-junior sentry-junior Bot commented Oct 20, 2026

Copy link
Copy Markdown
Contributor

📜 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:

  • Replaces the "We love pull requests from everyone" intro with a short welcome and a Discord link (discord.com/invite/sentry).
  • Adds a Submitting Changes section with:
    • a TL;DR: discuss in an issue first; PRs without prior discussion may be closed
    • Before You Start: find or create an issue, agree on the approach with a maintainer, check the assignee
    • Making 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 focused
    • AI 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

Mirrors getsentry/sentry-cocoa#9396 so the mobile/Java 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-pr workflow only posts an advisory comment.

💚 How did you test it?

Docs-only change; reviewed the rendered Markdown structure.

#skip-changelog

via Chris Aigner.

--

View Junior Session [Sentry]

Co-Authored-By: Chris Aigner <christoph.aigner@sentry.io>
@sentry

sentry Bot commented Oct 20, 2026

Copy link
Copy Markdown

📲 Install Builds

Android

🔗 App Name App ID Version Configuration
SDK Size io.sentry.tests.size 8.61.0 (1) release

⚙️ sentry-android Build Distribution Settings

This branch has not been deployed

No deployments
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant