mirror of
https://github.com/openwrt/luci.git
synced 2026-04-12 14:03:24 +04:00
luci: improvements to pull_request_template
As a new person to contributing on GitHub, I noticed the Pull Request template missing some important information that I would not have known until I tried my first Pull Request, such as the character limits and email requirements. I've seen others also get tripped up by this too, as they also were not aware until afterwards. So, I have tried to include the common ones into the template. I have also prepared a layout that would ensure a more uniform experience when looking through Pull Requests, with expected headings and the ability to provide the instruction of what to put under each heading. Minor grammar and punctuation edits as well. If the layout is preferred as a seperate Pull Request, let me know and I will do so. Will also update the CONTRIBUTING.md file to achieve the same goal as well. Signed-off-by: Joshua Criss <owlsy@outlook.com.au>
This commit is contained in:
committed by
Hannu Nyman
parent
c62d7fe850
commit
1bf35dff64
69
.github/pull_request_template.md
vendored
69
.github/pull_request_template.md
vendored
@@ -1,38 +1,61 @@
|
||||
<!--
|
||||
|
||||
Thank you for your contribution to the luci repository.
|
||||
Thank you for your contribution to the LuCI repository.
|
||||
|
||||
Please read this before creating your PR.
|
||||
/************************************************/
|
||||
/* PLEASE READ THIS BEFORE CREATING YOUR PR */
|
||||
/************************************************/
|
||||
|
||||
Review https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md
|
||||
especially if this is your first time to contribute to this repo.
|
||||
Review the Contributing Guidelines: https://github.com/openwrt/luci/blob/master/CONTRIBUTING.md
|
||||
(Especially if this is your first time to contribute to this repo.)
|
||||
|
||||
MUST NOT:
|
||||
- add a PR from your *main* branch - put it on a separate branch
|
||||
- add merge commits to your PR: rebase locally and force-push
|
||||
- Add a PR from your *main* branch - instead, put it on a separate branch.
|
||||
- Add merge commits to your PR - instead, rebase locally and force-push.
|
||||
|
||||
MUST:
|
||||
- increment any PKG_VERSION in the affected Makefile
|
||||
- set to draft if this PR depends on other PRs to e.g. openwrt/openwrt
|
||||
- each commit subject line starts with '<package name>: title'
|
||||
- each commit has a valid `Signed-off-by: ` (S.O.B.) with a reachable email
|
||||
- Increment any PKG_VERSION in the affected Makefile.
|
||||
- Set to draft if this PR depends on other PRs as well (e.g. openwrt/openwrt).
|
||||
- Have each commit subject line starting with '<package name>: title', and the title starting in lowercase, with a reasonable number of characters total.
|
||||
- Have a commit comment as it cannot be empty, with a reasonable number of characters per line.
|
||||
- Have each commit with a valid `Signed-off-by: ` (S.O.B.) with a reachable email (GitHub noreply emails are not accepted).
|
||||
* Forgot? `git commit --amend ; git push -f`
|
||||
* Tip: use `git commit --signoff`
|
||||
|
||||
MAY:
|
||||
- your S.O.B. *may* be a nickname
|
||||
- delete the below *optional* entries that do not apply
|
||||
- skip a `<package name>: title` first line subject if the commit is house-keeping or chore
|
||||
- Your S.O.B. *may* be a nickname.
|
||||
- Delete the *optional* entries in the checklist that do not apply.
|
||||
- Skip a `<package name>: title` first line subject if the commit is for house-keeping or a chore.
|
||||
|
||||
-->
|
||||
|
||||
- [ ] This PR is not from my *main* or *master* branch :poop:, but a *separate* branch :white_check_mark:
|
||||
- [ ] Each commit has a valid :black_nib: `Signed-off-by: <my@email.address>` row (via `git commit --signoff`)
|
||||
- [ ] Each commit and PR title has a valid :memo: `<package name>: title` first line subject for packages
|
||||
- [ ] Incremented :up: any `PKG_VERSION` in the Makefile
|
||||
- [ ] Tested on: (architecture, openwrt version, browser) :white_check_mark:
|
||||
- [ ] \( Preferred ) Mention: @ the original code author for feedback
|
||||
- [ ] \( Preferred ) Screenshot or mp4 of changes:
|
||||
- [ ] \( Optional ) Closes: e.g. openwrt/luci#issue-number
|
||||
- [ ] \( Optional ) Depends on: e.g. openwrt/packages#pr-number in sister repo
|
||||
- [ ] Description: (describe the changes proposed in this PR)
|
||||
## Pull Request details
|
||||
|
||||
### Description
|
||||
<!-- Describe the changes proposed in this PR. -->
|
||||
|
||||
|
||||
### Screenshot or video of changes _(If applicable)_
|
||||
<!-- Insert your screenshot or .mp4 here. -->
|
||||
|
||||
|
||||
### Maintainer _(Preferred)_
|
||||
<!-- You can find this by checking the history of the package Makefile. -->
|
||||
@<github-user>
|
||||
|
||||
---
|
||||
|
||||
## Tested on
|
||||
**OpenWrt version:**
|
||||
**LuCI version:**
|
||||
**Web browser:**
|
||||
|
||||
---
|
||||
|
||||
## Checklist
|
||||
- [ ] This PR is not from my *main* or *master* branch :poop:, but a *separate* branch. :white_check_mark:
|
||||
- [ ] Each commit has a valid :black_nib: `Signed-off-by: <my@email.address>` row (via `git commit --signoff`).
|
||||
- [ ] Each commit and PR title has a valid :memo: `<package name>: title` first line subject for packages.
|
||||
- [ ] Incremented :up: any `PKG_VERSION` in the Makefile.
|
||||
- [ ] _(Optional)_ Includes what Issue it closes (e.g. openwrt/luci#issue-number).
|
||||
- [ ] _(Optional)_ Includes what it depends on (e.g. openwrt/packages#pr-number in sister repo).
|
||||
|
||||
Reference in New Issue
Block a user