Skip to content

[CHORE] Remove GIT #280

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

Closed
wants to merge 2 commits into from
Closed

[CHORE] Remove GIT #280

wants to merge 2 commits into from

Conversation

valdezm
Copy link
Contributor

@valdezm valdezm commented Apr 30, 2025

No description provided.

ezyang and others added 2 commits April 18, 2025 18:57
codemcp makes does Git commits to keep track of writes and edits it makes to files, as well as to snapshot state of repo before and after it runs commands. Add a configuration option to codemcprc that optionally disables all of this, so that we NEVER issue any git commands automatically.

```git-revs
76f826f  (Base revision)
eaef646  Add git.enabled to default configuration
d42c38c  Add git_operations_enabled function to check if Git operations are enabled
3d05ab9  Import git_operations_enabled from config
4211fdc  Add check for disabled Git operations in create_commit_reference
274fe31  Update commit_changes docstring to mention git operations disabled case
d0ad8d4  Add check for disabled Git operations in commit_changes
ed91c17  Import git_operations_enabled from config
55c638f  Update check_git_tracking_for_existing_file to handle disabled Git operations
fac57d5  Add missing logging import
e9f9ee1  Fix duplicate logging import
ad2a7d6  Add git.enabled config option to README
f093df5  Auto-commit format changes
HEAD     Add end-to-end tests for Git operations disabled functionality
```

codemcp-id: 274-feat-add-config-option-to-disable-automatic-git-co
ghstack-source-id: 084ab9c
Pull-Request-resolved: ezyang#264
@valdezm
Copy link
Contributor Author

valdezm commented Apr 30, 2025

Sorry this was meant to be opened on my forked version of this project.

@valdezm valdezm closed this Apr 30, 2025
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.

3 participants