Revise pull request template for clarity and completeness#1626
Revise pull request template for clarity and completeness#1626Schamper merged 4 commits intofox-it:mainfrom
Conversation
Updated the pull request template to include new checklist items and clarify instructions for associating PRs with issues. Writing without comments should enforce contributors to more closely follow the desired standards.
|
@cochaviz thank you for your contribution! As this is your first code contribution, please read the following Contributor License Agreement (CLA). If you agree with the CLA, please reply with the following information:
Contributor License Agreement
Contribution License AgreementThis Contribution License Agreement ("Agreement") governs your Contribution(s) (as defined below) and conveys certain license rights to Fox-IT B.V. ("Fox-IT") for your Contribution(s) to Fox-IT"s open source Dissect project. This Agreement covers any and all Contributions that you ("You" or "Your"), now or in the future, Submit (as defined below) to this project. This Agreement is between Fox-IT B.V. and You and takes effect when you click an “I Accept” button, check box presented with these terms, otherwise accept these terms or, if earlier, when You Submit a Contribution.
|
|
@DissectBot agree |
Codecov Report✅ All modified and coverable lines are covered by tests. Additional details and impacted files@@ Coverage Diff @@
## main #1626 +/- ##
=======================================
Coverage 81.11% 81.11%
=======================================
Files 402 402
Lines 35244 35244
=======================================
Hits 28587 28587
Misses 6657 6657
Flags with carried forward coverage won't be shown. Click here to find out more. ☔ View full report in Codecov by Sentry. 🚀 New features to boost your workflow:
|
Revise pull request template for clarity and completeness
Currently, contributors might sometimes create unstructured PRs, making code review laborious for large changes. Often, the reviewer has to ask for context in a PR through the discussions, delaying merges.
The current PR template can be completely ignored because it's pure comments. If we use plaintext, contributors might be more inclined to complete the various checks.
Proposed Changes
Updated the pull request template to include new checklist items and clarify instructions for associating PRs with issues. Writing without comments should enforce contributors to more closely follow the desired standards.
Closes #1625
Checklist
related issue number