Update .gitignore to exclude unnecessary files#898
Update .gitignore to exclude unnecessary files#898mahdirahimi1999 wants to merge 2 commits intojazzband:masterfrom
Conversation
- Added *.pid and *.DS_Store to ignore process ID files and macOS system files.
for more information, see https://pre-commit.ci
| # macOS system files | ||
| *.DS_Store | ||
|
|
||
| # Process ID files |
There was a problem hiding this comment.
.DS_Store means Desktop Services Store. It's a hidden file that macOS creates in folders to save things like icon position, window size, and folder layout.
We add *.DS_Store to .gitignore because:
- It's not part of the code.
- It can be accidentally committed by macOS users.
- It adds unnecessary files to the project.
- It's not useful for people using Windows or Linux.
- macOS creates this file automatically when someone opens a folder, so it might get added without them noticing.
There was a problem hiding this comment.
.pid files store the Process ID of a running service or application (like a web server or background worker like celery). These files are temporary, system-specific, and should not be tracked in version control.
✅ Ignoring both helps keep the repository clean and free from unnecessary or machine-specific files.
There was a problem hiding this comment.
There was a problem hiding this comment.
I meant specifically PID. Where in this repository are we generating PID files? Seems unnecessary
There was a problem hiding this comment.
You're right, and I agree that adding *.pid is unnecessary. However, I wanted to point out that celerybeat.pid is already included in the .gitignore file under the Celery stuff section.
2 participants
Update .gitignore to Exclude Unnecessary Files
Description
This PR updates the
.gitignorefile to improve file exclusions and prevent unnecessary files from being tracked.Changes
*.pidand*.DS_Storeto ignore process ID files and macOS system files.Let me know if any adjustments are needed! 👍