Skip to content

chore(modules/claude-code): update readme #172

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
Jul 2, 2025

Conversation

hugodutka
Copy link
Contributor

The README points to an old version of claude code that doesn't support the --continue flag, making it incompatible with the latest version of the module. It's easy to copy and paste the example and run into an error. This PR updates the example version and adds troubleshooting notes.

@hugodutka hugodutka requested a review from matifali July 2, 2025 18:16
@hugodutka hugodutka force-pushed the hugodutka/claude-code-readme branch from 72d891d to ec0a53a Compare July 2, 2025 18:18
@matifali matifali requested a review from bcpeinhardt July 2, 2025 18:19
@matifali
Copy link
Member

matifali commented Jul 2, 2025

@bcpeinhardt do we still support updating the README without pushing a new version tag?

@hugodutka hugodutka enabled auto-merge (squash) July 2, 2025 18:20
@matifali
Copy link
Member

matifali commented Jul 2, 2025

@hugodutka, can you bump the patch version to ensure we include the latest changes? The registry only deploys from this repo on new tag pushes. Thanks

@matifali matifali added the version:patch Add to PRs requiring a patch version upgrade label Jul 2, 2025
@hugodutka hugodutka disabled auto-merge July 2, 2025 18:21
Copy link

github-actions bot commented Jul 2, 2025

❌ Version Bump Validation Failed

Bump Type: patch

Module versions need to be updated but haven't been bumped yet.

Required Actions:

  1. Run the version bump script locally: ./.github/scripts/version-bump.sh patch
  2. Commit the changes: git add . && git commit -m "chore: bump module versions (patch)"
  3. Push the changes: git push

Script Output:

🔍 Detecting modified modules...
Found modules:
registry/coder/modules/claude-code

📦 Processing: coder/claude-code
Found git tag: release/coder/claude-code/v2.0.0 (v2.0.0)
Current version: 2.0.0
New version: 2.0.1
Updating version references for coder/claude-code in registry/coder/modules/claude-code/README.md

🔧 Running formatter to ensure consistent formatting...

📋 Summary:
Bump Type: patch

Modules Updated:

- coder/claude-code: v2.0.0 → v2.0.1

READMEs Updated:

- coder/claude-code

✅ Version bump completed successfully!
📝 README files have been updated with new versions.

Next steps:
1. Review the changes: git diff
2. Commit the changes: git add . && git commit -m 'chore: bump module versions (patch)'
3. Push the changes: git push

Please update the module versions and push the changes to continue.

Copy link
Member

@matifali matifali left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM.

@hugodutka hugodutka changed the title chore(modules/claude-code): update readme chore(modules/claude-code): update readme Jul 2, 2025
@hugodutka hugodutka merged commit 121328f into main Jul 2, 2025
3 of 4 checks passed
@hugodutka hugodutka deleted the hugodutka/claude-code-readme branch July 2, 2025 18:26
@bcpeinhardt
Copy link
Collaborator

bcpeinhardt commented Jul 2, 2025

@bcpeinhardt do we still support updating the README without pushing a new version tag?

No we build the front end content entirely from the latest release now, that was the trade off we made to avoid the front end content being more up to date than the back end.

There's one exception: Module's that don't have a release in coder/registry (meaning they were last released when we were still using coder/modules) get their content built from the main branch of coder/registry, as it was copied over when we archived coder/modules so the source is exactly the same.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
version:patch Add to PRs requiring a patch version upgrade
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants