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.

Important

Depends on #711

⚡️ What's your motivation?

Improve CI performance.

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

Restore cache failed: Dependencies file is not found in /Users/runner/work/godog/godog. Supported file pattern: go.sum
Metric Previous Now Delta % Change
Total Workflow Runner Time 4m 7s
(247s)
Fastest Job 28s
Average Job 1m 1.2s
(61.2s)

🏷️ What kind of change is this?

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

♻️ Anything particular you want feedback on?

  • Requires fixing the broken godog main pipeline and merging into this branch
  • Subsequent benchmark should then be conducted prior to merge

📋 Checklist:

Copy link

Go API Changes

# summary%0AInferred base version: v0.15.1%0ASuggested version: v0.16.0

@kieran-ryan kieran-ryan self-assigned this Sep 14, 2025
@kieran-ryan kieran-ryan added the 🐛 bug Defect / Bug label Sep 14, 2025
@kieran-ryan kieran-ryan force-pushed the fix/cache-go-workflow branch 3 times, most recently from 21b4578 to 056beea Compare September 15, 2025 19:05
Must be specified as deviates from default `godog/godog/go.sum` path.
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.

1 participant