28 Commits

Author SHA1 Message Date
Avelino
f74f180683 Migrate PR quality checks from Node.js to Go (#6055)
* automate PR review: rewrite quality checks in Go and add diff validation

- Replace check-quality.js with Go implementation that adds:
  open source license detection, repo maturity (5+ months),
  CI/CD presence check, and README existence verification
- Add new check-pr-diff tool that validates README changes:
  single item per PR, link consistency with PR body,
  description format, non-promotional language, and
  category minimum size (3 items)
- Update pr-quality-check workflow to run both Go tools
  with separate quality and diff reporting sections
- Improve CONTRIBUTING.md with new "What is checked automatically"
  section documenting all blocking and non-blocking CI checks,
  plus what still requires manual maintainer review
- Improve PR template with clearer structure separating
  CI-validated requirements from manual review items

https://claude.ai/code/session_01JcdbDdtERhAXFFZmu3pCVY

* add actionable fix instructions to automated PR review comments

Each failed check now includes a "How to fix" section with specific
commands and steps the contributor needs to take, e.g.:
- missing go.mod → shows exact `go mod init` + `go mod tidy` commands
- missing semver release → shows `git tag v1.0.0` command and GitHub UI link
- low Go Report Card grade → shows `gofmt`, `go vet` commands
- missing PR body links → shows the exact text to paste
- description without period → shows the corrected entry line
- link mismatch → shows both URLs and which to update

https://claude.ai/code/session_01JcdbDdtERhAXFFZmu3pCVY

* add auto-merge when all PR quality checks pass

Adds an auto-merge job that enables GitHub's native auto-merge (squash)
after the quality and diff checks succeed. GitHub will only execute the
merge once ALL required branch protection status checks are green
(including the tests workflow).

Requires "Allow auto-merge" to be enabled in repo settings.

https://claude.ai/code/session_01JcdbDdtERhAXFFZmu3pCVY

* add workflow to re-run quality checks on all open PRs

Creates a manual workflow_dispatch workflow that iterates through all
open PRs and closes/reopens each one to re-trigger the quality check
pipeline. Includes a 2s delay between PRs to respect API rate limits.

Usage: Actions tab → "Re-check all open PRs" → Run workflow

https://claude.ai/code/session_01JcdbDdtERhAXFFZmu3pCVY

* skip quality checks for PRs that don't modify README.md

The quality checks (forge link, pkg.go.dev, Go Report Card, etc.) only
apply to package addition/removal PRs. Internal PRs (workflow changes,
docs, scripts) were failing because they naturally don't have those
fields in the PR body.

Now the workflow first detects if README.md is in the changed files:
- If yes: runs full quality + diff checks as before
- If no: posts a "Skipped" notice and passes cleanly

Fixes false failures like PR #6055.

https://claude.ai/code/session_01JcdbDdtERhAXFFZmu3pCVY

---------

Co-authored-by: Claude <noreply@anthropic.com>
2026-02-27 07:05:00 -03:00
oliverpool
73af87e792 Refactor PR template: hint to curate packages around the submission (#5233)
* refactor PR template

* typo
2024-03-08 15:14:25 +05:30
Tom Godkin
22c7595b8f Allow trunk-based workflows for awesome projects (#4976) 2023-08-15 23:37:23 +05:30
Avelino
979e55167a pr template: remove gocover.io
ref: 6ef5fb11ad
2023-01-12 22:24:01 -03:00
Kevin Mulvey
f59afbd692 correct spelling of 'details' in pr template (#4567) 2022-10-25 18:00:02 +05:30
Phani Rithvij
bb55de587f fix grammar 2022-06-01 18:23:35 +05:30
spekary
8821d6b649 Updating PR template to reflect new contribution guidelines 2022-03-02 14:46:16 -03:00
Avelino
31e63566db rename all hard ref branch master to main
Signed-off-by: Avelino <avelinorun@gmail.com>
2021-12-16 08:30:25 -03:00
Alex Archer
52b1da790f Update pull request template to be clearer for coverage service link section 2021-10-06 21:06:51 -03:00
Avelino
07773c8194 recovery pull request template to md
Signed-off-by: Avelino <avelinorun@gmail.com>
2021-07-15 07:24:36 -03:00
Avelino
3b249705dc Update and rename PULL_REQUEST_TEMPLATE.md to PULL_REQUEST_TEMPLATE.yaml 2021-07-15 07:11:22 -03:00
Sergey Podgornyy
52b1ea61cc Fix broken link to the contribution guidelines 2021-04-17 10:12:03 -03:00
Andy Pan
629908490d Fix the wrong URL of quality standards 2020-06-12 10:52:03 +08:00
Andy Pan
cff02f9ea2 Replace godoc.org with pkg.go.dev in contributing guideline 2020-03-14 11:57:09 +08:00
Avelino
597816f7af cover.run not exit more :/ 2019-07-01 17:35:32 -03:00
Avelino
60a55a4f00 upgrade cover.run example
new version used query string
2018-09-22 16:31:37 -03:00
Spencer
67e725b876 Added missing periods to ends of sentences. (#1535)
This change is done to keep things more consistent and have all
sentences end with periods. Previously, most sentences had periods
at the end, but not all.

This makes the document a little more professional looking and
raises the Awesome quality standard higher.
2017-08-13 00:18:26 -04:00
Willian Molinari
a4d360b36c Add a deadline for new PR on CONTRIBUTION (#1459)
We are trying to avoid having hundreds of open Pull Requests. By using
this rule we may close old PRs.
2017-06-16 16:08:08 -03:00
Thiago Avelino
52771083ee recommendation new coverage runner cover.run (#1400)
* recommendation new coverage runner cover.run
Shows how many percent that covered the package

* added link all covers sites
2017-05-01 18:53:00 -03:00
Kirill Danshin
d790cae6a0 add github link to links list + bold font weight 2016-07-21 03:17:09 +03:00
Kirill Danshin
04d0f0d91a rename coverage checkbox 2016-07-21 03:14:36 +03:00
Kirill Danshin
9935f12719 support any coverage service 2016-07-21 03:13:52 +03:00
Kirill Danshin
c6dc6c69b0 add more info to PR template and gocover.io altername 2016-06-22 11:01:31 +03:00
Dmitri Shuralyov
63f6cac77d Fix typo in PULL_REQUEST_TEMPLATE.md. 2016-04-19 17:27:22 -07:00
kirillDanshin
5c766a95a0 Update PULL_REQUEST_TEMPLATE.md
see #855
2016-04-03 22:53:20 +03:00
Dobrosław Żybort
f12e6787af Update PULL_REQUEST_TEMPLATE.md 2016-03-10 09:25:25 +01:00
Dobrosław Żybort
0214aa04d3 put text in PULL_REQUEST_TEMPLATE in one line 2016-03-08 08:31:12 +01:00
Dobrosław Żybort
da63591956 Create PULL_REQUEST_TEMPLATE.md
Mention #855
2016-03-07 11:33:46 +01:00