Skip to content

ci: sync with netresearch/.github templates/go-app - #556

Merged
CybotTM merged 2 commits into
mainfrom
sync/template-go-app-20260420-132948
Apr 20, 2026
Merged

ci: sync with netresearch/.github templates/go-app#556
CybotTM merged 2 commits into
mainfrom
sync/template-go-app-20260420-132948

Conversation

@CybotTM

@CybotTM CybotTM commented Apr 20, 2026

Copy link
Copy Markdown
Member

Auto-opened by sync-template.sh. Brings this repo back into alignment with the canonical go-app template in netresearch/.github.

To keep any diverging files, add their paths to .github/template.yaml's intentional-drift: list before merging — otherwise the next sync run will revert them.

Auto-generated by scripts/sync-template.sh. Any changes you want to keep
must be declared in .github/template.yaml's intentional-drift: list — the
check-template-drift.yml job will otherwise revert them on next sync.

Signed-off-by: Sebastian Mendel <info@sebastianmendel.de>
Copilot AI review requested due to automatic review settings April 20, 2026 11:29
@gemini-code-assist

Copy link
Copy Markdown

Note

Gemini is unable to generate a review for this pull request due to the file types involved not being currently supported.

@github-actions

github-actions Bot commented Apr 20, 2026

Copy link
Copy Markdown
Contributor

Dependency Review

✅ No vulnerabilities or license issues or OpenSSF Scorecard issues found.

Scanned Files

  • .github/workflows/codeql.yml

@github-actions github-actions Bot added the ci label Apr 20, 2026
github-actions[bot]
github-actions Bot previously approved these changes Apr 20, 2026

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval for maintainer PR

All automated quality gates passed. See SECURITY_CONTROLS.md for compensating controls.

Copilot AI left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Syncs this repository’s GitHub Actions workflows back to the canonical netresearch/.github go-app template, aligning CI/security automation with upstream defaults.

Changes:

  • Extend the container build workflow job permissions to include security-events: write.
  • Simplify CodeQL workflow by removing custom language detection and delegating language selection to the reusable workflow via languages: auto.

Reviewed changes

Copilot reviewed 2 out of 2 changed files in this pull request and generated no comments.

File Description
.github/workflows/container.yml Adds security-events: write permission to the container job (template alignment / security reporting support).
.github/workflows/codeql.yml Removes in-repo language detection job; uses template workflow with languages: auto.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@codecov

codecov Bot commented Apr 20, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 83.91%. Comparing base (ed95818) to head (3b87df6).
⚠️ Report is 3 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #556   +/-   ##
=======================================
  Coverage   83.91%   83.91%           
=======================================
  Files          17       17           
  Lines        1716     1716           
=======================================
  Hits         1440     1440           
  Misses        218      218           
  Partials       58       58           
Flag Coverage Δ
unittests 83.91% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

…rm64 only

Template default is 5 platforms (386/amd64/arm-v6/arm-v7/arm64) but
oven/bun and golang-alpine used in this repo's Dockerfile only publish
amd64+arm64 variants. Buildx fails with 'no match for platform in
manifest' on the other three.

Narrow to linux/amd64,linux/arm64 and record container.yml as
intentional-drift.

Signed-off-by: Sebastian Mendel <sebastian.mendel@netresearch.de>

@github-actions github-actions Bot left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Automated approval for maintainer PR

All automated quality gates passed. See SECURITY_CONTROLS.md for compensating controls.

@CybotTM
CybotTM merged commit 1954e1e into main Apr 20, 2026
26 checks passed
@CybotTM
CybotTM deleted the sync/template-go-app-20260420-132948 branch April 20, 2026 11:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants