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.

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

nobody needs a desktop.ini #4

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

casonclagg
Copy link

=P

@casonclagg
Copy link
Author

Neat project, btw

@Karosuo
Copy link

Karosuo commented Jul 5, 2020

Agree, those things may (and probably will) change over environments and setups, windows would create one if required automatically, therefore they do not belong to the repo.
But, also agree that this might be a minor problem, don't want to sound too perfectionist.
It is a very nice project.

@casonclagg
Copy link
Author

It's a joke/not-joke PR. Also, I added a .gitignore to keep future desktop.ini's from sneaking in...

Future work might include adding .DS_Store to the .gitignore

@Karosuo
Copy link

Karosuo commented Jul 6, 2020

You're right, I may be too serious, still getting used to the interaction.

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