Skip to content

fix(claude-code): run post-install script after configuring MCP and remove send-keys workaround #130

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 5 commits into from
Jun 2, 2025
Merged

fix(claude-code): run post-install script after configuring MCP and remove send-keys workaround #130

merged 5 commits into from
Jun 2, 2025

Conversation

bpmct
Copy link
Member

@bpmct bpmct commented Jun 1, 2025 •
edited
Loading

  • needed to install MCP servers
  • workaround sucked

@bpmct bpmct requested a review from DevelopmentCats June 1, 2025 21:01
@matifali matifali added the version:patch Add to PRs requiring a patch version upgrade label Jun 1, 2025
DevelopmentCats
DevelopmentCats approved these changes Jun 2, 2025
View reviewed changes
Copy link
Contributor

@DevelopmentCats DevelopmentCats left a comment

Choose a reason for hiding this comment

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

This has been tested and working.

Looks good to me as well!

@DevelopmentCats
Copy link
Contributor

@bpmct The only thing left is to update the versions in the README and Frontmatter

@DevelopmentCats DevelopmentCats self-requested a review June 2, 2025 23:16
@DevelopmentCats DevelopmentCats merged commit 9e7ce39 into main Jun 2, 2025
4 checks passed
@DevelopmentCats DevelopmentCats deleted the cc-fix-postinstall branch June 2, 2025 23:37
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@DevelopmentCats DevelopmentCats Awaiting requested review from DevelopmentCats

Assignees
No one assigned
Labels
version:patch Add to PRs requiring a patch version upgrade
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@bpmct @DevelopmentCats @matifali