Skip to content

Fix setup-go/checkout order and gitignore .claude/ - #97

Merged
marcelocantos merged 1 commit into
masterfrom
fix-workflow-step-order-and-gitignore
Apr 11, 2026
Merged

Fix setup-go/checkout order and gitignore .claude/#97
marcelocantos merged 1 commit into
masterfrom
fix-workflow-step-order-and-gitignore

Conversation

@marcelocantos

Copy link
Copy Markdown
Contributor

Two small follow-ups from PR #96:

  • go.yml: reorder steps so checkout runs before setup-go in all three
    jobs. actions/setup-go@v6 tries to restore the Go module cache by
    reading go.mod, but the previous order ran it before checkout, so
    go.mod wasn't on disk yet. The cache-restore emitted a non-fatal
    annotation on every run ("Dependencies file is not found"). With
    the correct order, the cache now restores successfully.

  • .gitignore: add .claude/ for Claude Code's per-project agent state.
    The directory has been sitting untracked in every working tree this
    session; adding it to .gitignore stops it showing up in git status.

Co-Authored-By: Claude Opus 4.6 (1M context) noreply@anthropic.com

Two small follow-ups from PR #96:

- go.yml: reorder steps so checkout runs before setup-go in all three
  jobs. actions/setup-go@v6 tries to restore the Go module cache by
  reading go.mod, but the previous order ran it before checkout, so
  go.mod wasn't on disk yet. The cache-restore emitted a non-fatal
  annotation on every run ("Dependencies file is not found"). With
  the correct order, the cache now restores successfully.

- .gitignore: add .claude/ for Claude Code's per-project agent state.
  The directory has been sitting untracked in every working tree this
  session; adding it to .gitignore stops it showing up in git status.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@marcelocantos
marcelocantos merged commit d95d20d into master Apr 11, 2026
3 checks passed
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