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

Ignore generated public resources #32

Merged
merged 4 commits into from
Jul 4, 2015

Conversation

petersandor
Copy link
Contributor

I believe these two directories are safe to ignore since they contain only resources generated by Gulp

@petersandor petersandor mentioned this pull request Jul 2, 2015
@timkurvers
Copy link
Member

Thanks! Could you order them alphabetically? Should public/templates be included here too?

@petersandor
Copy link
Contributor Author

Sure, I will order them. And you are right about the public/templates. I missed that one since that directory is included in repository. I might as well delete it, if you agree?

@timkurvers
Copy link
Member

Oh I see. I think the original idea was to at some point version control both scripts and styles too, but they're quite massive. Let's delete and ignore all three directories for now 👍

@timkurvers
Copy link
Member

Awesome stuff 👍

timkurvers added a commit that referenced this pull request Jul 4, 2015
Ignore generated public resources
@timkurvers timkurvers merged commit 1d5823b into wowserhq:master Jul 4, 2015
@petersandor petersandor deleted the ignore-generated branch July 4, 2015 14:47
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

2 participants