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

.gitignore changes: Thumbs.db and public directory (with all subdirectories and files). #283

Merged
merged 2 commits into from Mar 10, 2016

Conversation

zacanger
Copy link
Contributor

@zacanger zacanger commented Mar 9, 2016

Addresses #
Does not address any open issue; just addresses potential confusion over what should/shouldn't be ignored in public, and catches all Thumbs.db files.

Summary of changes:
Previously had many files/subdirectories broken out in the .gitignore; seems the idea is just to ignore the entire directory, which can be done with just the one line.
Not specifying subdirectory for Thumbs.db files will let the gitignore just catch all of 'em.

Previously had many files/subdirectories broken out in the `.gitignore`; seems the idea is just to ignore the entire directory, which can be done with just the one line.

Does not address any open issue; just addresses potential confusion over what should/shouldn't be ignored in `public`.
Not specifying any directory means it'll catch all `Thumbs.db` files in any directory.
@geoffp
Copy link
Contributor

geoffp commented Mar 10, 2016

Thanks for this, @zacanger. @bmuenzenmeyer, I gave this a quick test, and it seems to work. Any objections?

@bmuenzenmeyer
Copy link
Member

No objections, as long as a base install works. I love the notion of cleaning stuff up like this. This ignore file is another place that showcases how ~4 years of hacking on the repo have evolved.

@geoffp
Copy link
Contributor

geoffp commented Mar 10, 2016

Groovy

geoffp added a commit that referenced this pull request Mar 10, 2016
`.gitignore` changes: `Thumbs.db` and `public` directory (with all subdirectories and files).
@geoffp geoffp merged commit 7ab342c into pattern-lab:dev Mar 10, 2016
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.

None yet

3 participants