Skip to content

feat(release)!: orchestrate backmerge via backmerge-sync; expose go-pr-validation inputs & enforce coverage #428

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
bedatty merged 8 commits into from
Jun 17, 2026
Merged

feat(release)!: orchestrate backmerge via backmerge-sync; expose go-pr-validation inputs & enforce coverage #428

Show file tree
Hide file tree
Changes from 2 commits
Commits
Show all changes
8 commits
Select commit Hold shift + click to select a range
fd9e770
feat(go-pr-validation): expose golangci_lint_args and app_name_prefix
bedatty Jun 17, 2026
78419ae
fix(go-pr-analysis): map golangci_lint_args via env to prevent shell …
bedatty Jun 17, 2026
a3554fa
fix(go-pr-analysis): split golangci_lint_args via read -ra to satisfy…
bedatty Jun 17, 2026
94b8be9
feat(go-pr-analysis)!: enforce coverage threshold by default
bedatty Jun 17, 2026
6a5167b
feat(release)!: orchestrate backmerge via backmerge-sync instead of s…
bedatty Jun 17, 2026
bb4a360
fix(release): drop trailing blank line in .releaserc.yml (yamllint em…
bedatty Jun 17, 2026
06b0e6c
feat(release): add dry_run input and preflight-validate backmerge config
bedatty Jun 17, 2026
ce94449
feat(build)!: default enable_ghcr to true in go-release and build
bedatty Jun 17, 2026
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
4 changes: 3 additions & 1 deletion .github/workflows/go-pr-analysis.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -213,7 +213,9 @@ jobs:
- name: Run GolangCI-Lint (direct)
if: steps.detect-make.outputs.use_make != 'true'
working-directory: ${{ matrix.app.working_dir }}
run: golangci-lint run ${{ inputs.golangci_lint_args }}
env:
GOLANGCI_LINT_ARGS: ${{ inputs.golangci_lint_args }}
run: golangci-lint run $GOLANGCI_LINT_ARGS

# ============================================
# SECURITY SCANNING
Expand Down
10 changes: 10 additions & 0 deletions .github/workflows/go-pr-validation.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -92,6 +92,14 @@ on:
description: 'GolangCI-Lint version'
type: string
default: 'v1.62.2'
golangci_lint_args:
description: 'Extra arguments passed to golangci-lint (e.g. --timeout=5m)'
type: string
default: '--timeout=5m'
app_name_prefix:
description: 'Prefix used to namespace coverage/build artifacts'
type: string
default: ''
Comment thread
coderabbitai[bot] marked this conversation as resolved.
coverage_threshold:
description: 'Minimum coverage percentage required (0-100)'
type: number
Expand Down Expand Up @@ -208,6 +216,8 @@ jobs:
runner_type: ${{ inputs.runner_type }}
go_version: ${{ inputs.go_version }}
golangci_lint_version: ${{ inputs.golangci_lint_version }}
golangci_lint_args: ${{ inputs.golangci_lint_args }}
app_name_prefix: ${{ inputs.app_name_prefix }}
coverage_threshold: ${{ inputs.coverage_threshold }}
fail_on_coverage_threshold: ${{ inputs.fail_on_coverage_threshold }}
go_private_modules: ${{ inputs.go_private_modules }}
Expand Down
2 changes: 2 additions & 0 deletions docs/go-pr-validation.md
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -38,6 +38,8 @@ The `go-analysis`, `security` and `lib-version` pipelines each have a `*-gate` a
| `target_branches_for_source_check` | Target branches requiring source validation | string | `main` |
| `go_version` | Go version | string | `1.23` |
| `golangci_lint_version` | GolangCI-Lint version | string | `v1.62.2` |
| `golangci_lint_args` | Extra arguments passed to golangci-lint (e.g. `--timeout=5m`) | string | `--timeout=5m` |
| `app_name_prefix` | Prefix used to namespace coverage/build artifacts | string | `''` |
| `coverage_threshold` | Minimum coverage percentage (0-100) | number | `80` |
| `fail_on_coverage_threshold` | Fail when coverage is below threshold | boolean | `false` |
| `go_private_modules` | GOPRIVATE pattern for private modules | string | `''` |
Expand Down