Skip to content

feat: update Python.gitignore to exclude PDF, CSV, and temporary files#4869

Open
andresm-data wants to merge 2 commits into
github:mainfrom
andresm-data:feat/binary_documents_python
Open

feat: update Python.gitignore to exclude PDF, CSV, and temporary files#4869
andresm-data wants to merge 2 commits into
github:mainfrom
andresm-data:feat/binary_documents_python

Conversation

@andresm-data

Copy link
Copy Markdown

Link to the application or project's homepage

https://www.python.org/

Reasons for making this change

These changes add common generated files that should not be versioned, specifically PDF outputs and temporary Microsoft Office files. These files are usually build artifacts or temporary lock files, add noise to commits, and are not source files that should be tracked.

Links to documentation supporting these rule changes

Merge and Approval Steps

Copilot AI review requested due to automatic review settings June 11, 2026 18:06
@andresm-data andresm-data requested a review from a team as a code owner June 11, 2026 18:06

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Pull request overview

Note

Copilot was unable to run its full agentic suite in this review.

Updates the repository’s Python .gitignore rules to exclude additional generated artifacts, preventing accidental commits of exported data and temporary Office files.

Changes:

  • Ignore generated document/data export files (*.pdf, *.csv)
  • Ignore Microsoft Office temporary files (~$*)

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

Comment thread Python.gitignore Outdated
Comment on lines +39 to +41
# Generated documents / data exports
*.pdf
*.csv

Copy link
Copy Markdown
Author

Choose a reason for hiding this comment

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

I have already applied the requested changes. I will be attentive to any further corrections.

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