Skip to content

AI/LLMs: Claude is an idiot and so am I#4738

Merged
dirkhh merged 1 commit intomasterfrom
claude
Mar 2, 2026
Merged

AI/LLMs: Claude is an idiot and so am I#4738
dirkhh merged 1 commit intomasterfrom
claude

Conversation

@dirkhh
Copy link
Collaborator

@dirkhh dirkhh commented Mar 2, 2026

I trusted that Claude would give me a correct workflow. I guess I could have seen this issue if I had thought about it harder -- or more likely, if I had looked at how other repos have this set up.

The Claude action doesn't check out the repo. We need to do this ourselves, first.

Oh well. Second try.

Describe the pull request:

  • Bug fix
  • Build system change

Mentions:

I'll just merge it, I guess.

I trusted that Claude would give me a correct workflow. I guess I could have
seen this issue if I had thought about it harder -- or more likely, if I had
looked at how other repos have this set up.

The Claude action doesn't check out the repo. We need to do this ourselves,
first.

Oh well. Second try.

Signed-off-by: Dirk Hohndel <dirk@hohndel.org>
@github-actions
Copy link

github-actions bot commented Mar 2, 2026

Artifacts:
Subsurface-Documentation-6.0.5554-patch.1.pull-request.claude
WARNING: Use at your own risk.

@dirkhh dirkhh merged commit aff0455 into master Mar 2, 2026
27 of 28 checks passed
@mikeller
Copy link
Member

mikeller commented Mar 2, 2026

The Claude action doesn't check out the repo. We need to do this ourselves, first.

I am actually surprised that it has to - it'll need way more context than just the current state, like the previous state, comments, ..., so checking out isn't giving it nowhere near the context it needs.
From the actual error comment I am not sure if the checkout is strictly needed, or if this is a chicken and egg situation.

@dirkhh
Copy link
Collaborator Author

dirkhh commented Mar 2, 2026

I honestly don't know. I'm running out of "hacking time" here... so I may give this one a pause. Let's see if the last attempt worked... after this I'll switch to a test repo and try it there without all the unintended side effects of this "production repo"

@dirkhh
Copy link
Collaborator Author

dirkhh commented Mar 2, 2026

hmm... this may be an issue that my personal Claude account cannot be used with an org repo.
I need to reach out to a friend at Anthropic, I guess, to figure out if this can be made to work...

@mikeller
Copy link
Member

mikeller commented Mar 2, 2026

I honestly don't know. I'm running out of "hacking time" here... so I may give this one a pause. Let's see if the last attempt worked... after this I'll switch to a test repo and try it there without all the unintended side effects of this "production repo"

In order to save time when working on 'explorative' CICD changes, feel free to push directly into master - we can always do a pro-forma pull request later if the changes need socialising.
And yes, 'explorativeness' is a feature of 'regular' changes that tends to jump out at you.

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.

2 participants