Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Repo contains OS X DS_Store files #7

Closed
tallzilla opened this issue Jan 23, 2024 · 2 comments · May be fixed by #8
Closed

Repo contains OS X DS_Store files #7

tallzilla opened this issue Jan 23, 2024 · 2 comments · May be fixed by #8

Comments

@tallzilla
Copy link

tallzilla commented Jan 23, 2024 •
edited
Loading

I love this project, I've been playing around with this for several hours and look forward to the v2 waveshare drivers.

One small nit - this repo contains .DS_Store and ._.DS_Store files in each directory, presumably written by the OS X filesystem you're working on (this happens to me all the time). If you add these to a global .gitignore file they won't be passed on for everyone to see.

This will add a global ignore to your local projects
git config --global core.excludesfile "~/.gitignore" && echo *.DS_Store >> ~/.gitignore

Thanks!
@tallzilla
Copy link
Author

tallzilla commented Jan 23, 2024 •
edited
Loading

Actually the above command just future-proofs your code. You'll need to run this from your project's root directory to remove the existing DS_Store files in the project:

find . -name .DS_Store -print0 | xargs -0 git rm --ignore-unmatch
find . -name ._.DS_Store -print0 | xargs -0 git rm --ignore-unmatch

tallzilla added a commit to tallzilla/zerowriter1 that referenced this issue Jan 23, 2024
@tallzilla
Removed DS_Store files
a6e3fee 
Fixes zerowriter#7
This was referenced Jan 23, 2024
Open
Closed
@zerowriter
Copy link
Owner

Thanks! I will add this. I frequently work between OSX, linux, and PC so -- kind of all over the place.

zerowriter pushed a commit that referenced this issue May 6, 2024
@Attomsk
Fixed issues #5 #6 #7
cde72c2
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Removed DS_Store files tallzilla/zerowriter1
2 participants
@tallzilla @zerowriter