Skip to content

luci: minor edits to CONTRIBUTING.md and PR template#8547

Merged
systemcrash merged 1 commit intoopenwrt:masterfrom
owlsy:minor-edit
Apr 16, 2026
Merged

luci: minor edits to CONTRIBUTING.md and PR template#8547
systemcrash merged 1 commit intoopenwrt:masterfrom
owlsy:minor-edit

Conversation

@owlsy
Copy link
Copy Markdown
Contributor

@owlsy owlsy commented Apr 12, 2026

Pull Request details

Description

For PR template:

  • Minor fix to 'please read' section, as symbol spacing was different on GitHub website than on GitHub Desktop.
  • Added HTML comment with example on how to complete the checklist using [x].
  • Added examples to the 'tested on' section.
  • Added abbreviation explanation for PR to better understand terminology, similar to using S.O.B.

For CONTRIBUTING.md:

  • Bumped up release branch example.
  • Minor capitalisation fixes for PR.
  • Minor formatting fixes.

Checklist

  • This PR is not from my main or master branch 💩, but a separate branch. ✅
  • Each commit has a valid ✒️ Signed-off-by: <my@email.address> row (via git commit --signoff).
  • Each commit and PR title has a valid 📝 <package name>: title first line subject for packages.

Comment thread .github/pull_request_template.md Outdated
For PR template, minor fix to 'please read'
section, as symbol spacing was different
on GitHub website than on GitHub Desktop.
Added HTML comment with example on how
to complete the checklist using [x]. Added
examples to the 'tested on' section.
Added abbreviation explanation for PR to
better understand terminology, similar to
using S.O.B. For CONTRIBUTING.md,
bumped up release branch example, and
minor capitalisation fixes for PR.

Signed-off-by: Joshua Criss <owlsy@outlook.com.au>
@systemcrash systemcrash merged commit 5552cf2 into openwrt:master Apr 16, 2026
4 checks passed
@systemcrash
Copy link
Copy Markdown
Contributor

OK

@owlsy owlsy deleted the minor-edit branch April 17, 2026 06:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants