Skip to content

ci: optimize GitHub Actions workflows#159

Merged
jongio merged 2 commits intomainfrom
fix/optimize-github-actions
Mar 12, 2026
Merged

ci: optimize GitHub Actions workflows#159
jongio merged 2 commits intomainfrom
fix/optimize-github-actions

Conversation

@jongio
Copy link
Copy Markdown
Owner

@jongio jongio commented Mar 12, 2026

Changes

  • Enable Go module caching across all workflows (saves 30-90s per run)
  • Add concurrency groups to prevent duplicate CI runs on rapid pushes
  • Remove redundant Go bootstrap pattern (was installing Go twice)
  • Add job timeouts to prevent runaway costs (default is 6 hours!)
  • Cache Go development tools (golangci-lint, gosec, etc.)
  • Remove duplicate lint jobs where preflight already runs linting

Estimated savings: ~2-5 minutes per CI run

- Enable Go module caching across all workflows
- Add concurrency groups to prevent duplicate runs
- Remove redundant Go bootstrap pattern
- Add job timeouts to prevent runaway costs
- Cache Go development tools
- Remove duplicate lint jobs

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
@github-actions
Copy link
Copy Markdown
Contributor

github-actions bot commented Mar 12, 2026

🚀 Website Preview

Your PR preview was available here.

Preview has been cleaned up as the PR was closed.

github-actions bot added a commit that referenced this pull request Mar 12, 2026
@codecov
Copy link
Copy Markdown

codecov bot commented Mar 12, 2026

Codecov Report

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

Additional details and impacted files
@@           Coverage Diff           @@
##             main     #159   +/-   ##
=======================================
  Coverage   54.86%   54.87%           
=======================================
  Files         147      147           
  Lines       22283    22283           
=======================================
+ Hits        12226    12228    +2     
+ Misses       9286     9284    -2     
  Partials      771      771           
Flag Coverage Δ
unittests 54.87% <ø> (+<0.01%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.
see 1 file with indirect coverage changes

🚀 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.

Go 1.26.0 has known stdlib vulnerabilities (GO-2026-4599,
GO-2026-4600, GO-2026-4601, GO-2026-4602) fixed in Go 1.26.1.
This was causing govulncheck failures in CI.

Co-authored-by: Copilot <223556219+Copilot@users.noreply.github.com>
github-actions bot added a commit that referenced this pull request Mar 12, 2026
@github-actions
Copy link
Copy Markdown
Contributor

🚀 Test This PR

A preview build (0.13.2-pr159) is ready for testing!

🌐 Website Preview

Live Preview: https://jongio.github.io/azd-app/pr/159/

One-Line Install (Recommended)

PowerShell (Windows):

iex "& { $(irm https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/install-pr.ps1) } -PrNumber 159 -Version 0.13.2-pr159"

Bash (macOS/Linux):

curl -fsSL https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/install-pr.sh | bash -s 159 0.13.2-pr159

Uninstall

When you're done testing:

PowerShell (Windows):

iex "& { $(irm https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/uninstall-pr.ps1) } -PrNumber 159"

Bash (macOS/Linux):

curl -fsSL https://raw.githubusercontent.com/jongio/azd-app/main/cli/scripts/uninstall-pr.sh | bash -s 159

Build Info:

What to Test:
Please review the PR description and test the changes described there.

@jongio jongio merged commit c3e6d94 into main Mar 12, 2026
15 checks passed
github-actions bot added a commit that referenced this pull request Mar 12, 2026
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.

1 participant