diff --git a/.github/pull_request_template.md b/.github/pull_request_template.md index 15fbc1c..7345962 100644 --- a/.github/pull_request_template.md +++ b/.github/pull_request_template.md @@ -1,32 +1,69 @@ -# Description +# PR Summary -## Summary +Sci/Tech Reviewer: +Code Reviewer: -_Briefly describe the feature being introduced._ + -## Changes + -_List the major changes made in this pull request._ + -## Dependency + -_List dependent changes. Can use build-group logic here._ +## Code Quality Checklist -## Impact +- [ ] I have performed a self-review of my own code +- [ ] My code follows the project's [style guidelines](https://metoffice.github.io/lfric_core/how_to_contribute/index.html#how-to-contribute-index) +- [ ] Comments have been included that aid understanding and enhance the readability of the code +- [ ] My changes generate no new warnings +- [ ] All automated checks in the CI pipeline have completed successfully -_Discuss any potential impacts this feature may have on existing functionalities._ +## Testing -## Issues addressed +- [ ] This change has been tested appropriately (please describe) -Resolves +## Security Considerations -_List issue(s) related to this PR._ +- [ ] I have reviewed my changes for potential security issues +- [ ] Sensitive data is properly handled (if applicable) +- [ ] Authentication and authorisation are properly implemented (if applicable) -## Coordinated merge +## AI Assistance and Attribution -_Specify any coordinated merges here._ +- [ ] Some of the content of this change has been produced with the assistance of _Generative AI tool name_ (e.g., Met Office Github Copilot Enterprise, Github Copilot Personal, ChatGPT GPT-4, etc) and I have followed the [Simulation Systems AI policy](https://metoffice.github.io/simulation-systems/FurtherDetails/ai.html) (including attribution labels) + -## Checklist +# Sci/Tech Review + + + + +- [ ] I understand this area of code and the changes being added +- [ ] The proposed changes correspond to the pull request description +- [ ] Documentation is sufficient (do documentation papers need updating) +- [ ] Sufficient testing has been completed + +(_Please alert the code reviewer via a tag when you have approved the SR_) + +# Code Review + + + +- [ ] All dependencies have been resolved +- [ ] Related Issues have been properly linked and addressed +- [ ] Code quality standards have been met +- [ ] Tests are adequate and have passed +- [ ] Security considerations have been addressed +- [ ] Performance impact is acceptable -- [ ] I have performed a self-review of my own changes diff --git a/.github/workflows/track-review-project.yaml b/.github/workflows/track-review-project.yaml new file mode 100644 index 0000000..639477c --- /dev/null +++ b/.github/workflows/track-review-project.yaml @@ -0,0 +1,17 @@ +name: Track Review Project + +on: + workflow_run: + workflows: [Trigger Review Project] + types: + - completed + +permissions: + actions: read + contents: read + pull-requests: write + +jobs: + track_review_project: + uses: MetOffice/growss/.github/workflows/track-review-project.yaml@main + secrets: inherit diff --git a/.github/workflows/trigger-project-workflow.yaml b/.github/workflows/trigger-project-workflow.yaml new file mode 100644 index 0000000..4d0a8f6 --- /dev/null +++ b/.github/workflows/trigger-project-workflow.yaml @@ -0,0 +1,17 @@ +name: Trigger Review Project + +on: + pull_request_target: + types: ["opened", "synchronize", "reopened", "edited", "review_requested", "review_request_removed"] + pull_request_review: + pull_request_review_comment: + +permissions: + actions: read + contents: read + pull-requests: write + +jobs: + trigger_project_workflow: + uses: MetOffice/growss/.github/workflows/trigger-project-workflow.yaml@main + secrets: inherit