Skip to content

Conversation

kieran-ryan
Copy link
Member

@kieran-ryan kieran-ryan commented Sep 14, 2025

🤔 What's changed?

Explicit go lock file path to enable caching in GitHub workflow.

⚡️ What's your motivation?

Improve CI performance.

Addressing that the setup-go action expects the lock file at messages/message/go.sum (see #50333999175 logs) rather than messages/go/go.sum.

Restore cache failed: Dependencies file is not found in /Users/runner/work/messages/messages. Supported file pattern: go.sum
Metric Previous Now Delta % Change
Total Workflow Runner Time 2m 13s
(133s)
1m 34s
(94s)
-39s -29.3%
Fastest Job 24s 12s -12s -50%
Average Job 33.3s 23.5s -9.8s -29.4%

🏷️ What kind of change is this?

  • 🐛 Bug fix (non-breaking change which fixes a defect)

📋 Checklist:

@kieran-ryan kieran-ryan self-assigned this Sep 14, 2025
@kieran-ryan kieran-ryan added the 🐛 bug Defect / Bug label Sep 14, 2025
@codecov-commenter
Copy link

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 100.00%. Comparing base (5cf3d14) to head (d0ce69e).
⚠️ Report is 19 commits behind head on main.

Additional details and impacted files
@@            Coverage Diff            @@
##              main      #334   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files            3         3           
  Lines          200       200           
=========================================
  Hits           200       200           

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

Must be specified as deviates from default `messages/message/go.sum` path.
@kieran-ryan kieran-ryan marked this pull request as ready for review September 14, 2025 18:00
@mpkorstanje mpkorstanje merged commit de6e9cc into main Sep 14, 2025
37 checks passed
@mpkorstanje mpkorstanje deleted the fix/cache-go-workflow branch September 14, 2025 19:47
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
🐛 bug Defect / Bug
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants