-
Notifications
You must be signed in to change notification settings - Fork 1.1k
Update contribution guidelines #6869
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
Update contribution guidelines #6869
Conversation
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## dev-2.x #6869 +/- ##
=============================================
- Coverage 72.10% 72.10% -0.01%
+ Complexity 19671 19669 -2
=============================================
Files 2125 2125
Lines 79533 79533
Branches 8051 8051
=============================================
- Hits 57349 57345 -4
- Misses 19347 19349 +2
- Partials 2837 2839 +2 ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
|
The decisions from PR #6866 will have to be referenced in |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This is nicely written and adds much more needed clarity to the expectations for how issues and PRs will be dealt with going forward.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you take a look at Binh's comments, I'm happy to review this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
If you take a look at Binh's comments, I'm happy to review this.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
This does not count as a review, but I will file a few suggestions.
In general consider to format the document using 100 characters long lines. This is mainly to make it easy to read in the "source" view.
Summary
This PR updates the contributing guidelines as discussed in the September 2025 OTP Summit in Helsinki.
Issue
N/A
Unit tests
N/A
Documentation
CONTRIBUTING.mdREADME.md.github/pull_request_template.md