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 cucumber_expressions/cucumber_expressions/go.sum (see #17709743808 logs) rather than cucumber_expressions/go/go.sum.

Restore cache failed: Dependencies file is not found in /Users/runner/work/cucumber_expressions/cucumber_expressions. Supported file pattern: go.sum
Metric Previous Now Delta % Change
Total Workflow Runner Time 1m 33s
(99s)
1m 19s
(79s)
-20s -20.2%
Fastest Job 18s 17s -1s -5.6%
Average Job 31s 26.3s -4.7s -15.2%

🏷️ 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
Must be specified as deviates from default `cucumber-expressions/cucumber-expressions/go.sum` path.
@kieran-ryan kieran-ryan marked this pull request as ready for review September 14, 2025 20:15
@mpkorstanje mpkorstanje merged commit d94eb8f into main Sep 14, 2025
30 checks passed
@mpkorstanje mpkorstanje deleted the fix/cache-go-workflow branch September 14, 2025 20:33
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.

2 participants