-
Notifications
You must be signed in to change notification settings - Fork 31
FIX: Retain .gdn files during auto sync #379
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
base: main
Are you sure you want to change the base?
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Pull request overview
This PR updates the GitHub-to-Azure DevOps sync pipeline to preserve the .gdn directory during the daily synchronization process. The .gdn directory contains Guardian security scanning baseline and suppression files (.gdnbaselines and .gdnsuppress) that are essential for the OneBranch SDL (Security Development Lifecycle) compliance and should not be synced from GitHub since they are specific to the Azure DevOps environment.
Key Changes:
- Modified
git rmcommand to exclude.gdndirectory from deletion during sync - Modified
git checkoutcommand to exclude.gdndirectory from being overwritten with GitHub content
💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.
| git fetch github main | ||
| git rm -rf . | ||
| git checkout github/main -- . | ||
| git rm -rf . -- :!.gdn |
Copilot
AI
Dec 17, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
The double-dash separator -- is in the wrong position for the git rm command. The correct syntax should be git rm -rf -- . :!.gdn with the -- before the pathspecs, not between them. The -- separator tells git that everything after it should be treated as pathspecs, not options. Without proper placement, the command may not correctly interpret :!.gdn as an exclusion pattern.
| git rm -rf . -- :!.gdn | |
| git rm -rf -- . :!.gdn |
📊 Code Coverage Report
Diff CoverageDiff: main...HEAD, staged and unstaged changesNo lines with coverage information in this diff. 📋 Files Needing Attention📉 Files with overall lowest coverage (click to expand)mssql_python.pybind.logger_bridge.hpp: 58.8%
mssql_python.pybind.logger_bridge.cpp: 59.2%
mssql_python.row.py: 66.2%
mssql_python.helpers.py: 67.5%
mssql_python.pybind.ddbc_bindings.cpp: 69.3%
mssql_python.pybind.connection.connection.cpp: 73.6%
mssql_python.ddbc_bindings.py: 79.6%
mssql_python.connection.py: 83.9%
mssql_python.cursor.py: 84.4%
mssql_python.__init__.py: 84.9%🔗 Quick Links
|
Work Item / Issue Reference
Summary
This pull request updates the GitHub-Azure DevOps sync pipeline to improve handling of the
.gdndirectory during repository synchronization. The main change ensures that the.gdndirectory is preserved when files are removed and checked out from themainbranch.File sync improvements:
git rm -rf .andgit checkout github/main -- .commands to exclude the.gdndirectory, preventing it from being deleted or overwritten during the sync process.