From e12dd1e7a6cec1862133a6d8459820f40bb4d5f9 Mon Sep 17 00:00:00 2001 From: Stefan Raus Date: Thu, 24 Aug 2023 16:05:07 +0100 Subject: [PATCH] Add PR Template and Code Owners Code owners are automatically added as reviewers when someone opens a PR. Other reviewers can be added beside code owners. PR Template automatically populates the body of new Pull Requests with the content of PULL_REQUEST_TEMPLATE.md. Signed-off-by: Stefan Raus --- .github/CODEOWNERS | 11 +++++++++++ .github/PULL_REQUEST_TEMPLATE.md | 19 +++++++++++++++++++ .gitignore | 2 ++ 3 files changed, 32 insertions(+) create mode 100644 .github/CODEOWNERS create mode 100644 .github/PULL_REQUEST_TEMPLATE.md diff --git a/.github/CODEOWNERS b/.github/CODEOWNERS new file mode 100644 index 00000000000..6705d6dd0af --- /dev/null +++ b/.github/CODEOWNERS @@ -0,0 +1,11 @@ +# This file contains code owners for No-OS Pull Requests to master/main. +# Code owners are added automatically as reviewers. All PRs open against +# master/main will require approval of at least one code owner. +# +# More general documentation about code owners can be found here: +# https://github.blog/2017-07-06-introducing-code-owners/ + +# CODEOWNERS file format: + + +##### Global code owners (for whole files from repo) ##### +* darius.berghe@analog.com ciprian.regus@analog.com dragos.bogdan@analog.com antoniu.miclaus@analog.com ramona.gradinariu@analog.com george.mois@analog.com diff --git a/.github/PULL_REQUEST_TEMPLATE.md b/.github/PULL_REQUEST_TEMPLATE.md new file mode 100644 index 00000000000..635567bf242 --- /dev/null +++ b/.github/PULL_REQUEST_TEMPLATE.md @@ -0,0 +1,19 @@ +## Pull Request Description + +Please replace this with a detailed description and motivation of the changes. +You can tick the checkboxes below with an 'x' between square brackets or just check them after publishing the PR. +If this PR contains a breaking change, list dependent PRs and try to push all related PRs at the same time. + +## PR Type +- [ ] Bug fix (change that fixes an issue) +- [ ] New feature (change that adds new functionality) +- [ ] Breaking change (has dependencies in other repos or will cause CI to fail) + +## PR Checklist +- [ ] I have followed the [Coding style guidelines](http://analogdevicesinc.github.io/no-OS/drivers_guide.html#coding-style) +- [ ] I have performed a self-review of the changes +- [ ] I have commented my code, at least hard-to-understand parts +- [ ] I have build all projects affected by the changes in this PR +- [ ] I have tested in hardware affected projects, at the relevant boards +- [ ] I have signed off all commits from this PR +- [ ] I have updated the documentation (wiki pages, ReadMe etc), if applies diff --git a/.gitignore b/.gitignore index f3a97d4a46f..69ceefa1b8a 100644 --- a/.gitignore +++ b/.gitignore @@ -21,6 +21,8 @@ aducm_project aducm_build .vscode .Xil +.github/CODEOWNERS +.github/PULL_REQUEST_TEMPLATE.md *.a *.o