Skip to content

fix: update git_config to use correct parameter name#106

Merged
patrick-stephens merged 2 commits intomainfrom
fix_config_dir_git_config_plugin
Mar 19, 2026
Merged

fix: update git_config to use correct parameter name#106
patrick-stephens merged 2 commits intomainfrom
fix_config_dir_git_config_plugin

Conversation

@patrick-stephens
Copy link
Contributor

clone_path should be config_dir.

Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
@vercel
Copy link

vercel bot commented Mar 19, 2026

The latest updates on your projects. Learn more about Vercel for GitHub.

Project Deployment Actions Updated (UTC)
documentation Ready Ready Preview, Comment Mar 19, 2026 3:09pm

Request Review

Copy link

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Updates the git_config plugin documentation to reflect the correct configuration parameter name (config_dir instead of clone_path) so users configure the feature correctly.

Changes:

  • Renames the documented parameter from clone_path to config_dir across option tables, examples, and path placeholders.
  • Moves/expands the debug logging guidance within the Troubleshooting section.
  • Updates troubleshooting/security references to use {config_dir} consistently.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

You can also share your feedback on Copilot code review. Take the survey.

Signed-off-by: Patrick Stephens <pat@telemetryforge.io>
@patrick-stephens patrick-stephens merged commit 33f9ba1 into main Mar 19, 2026
7 checks passed
@patrick-stephens patrick-stephens deleted the fix_config_dir_git_config_plugin branch March 19, 2026 15:10
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