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

Update README.md #215

Merged
merged 1 commit into from
Feb 1, 2014
Merged

Update README.md #215

merged 1 commit into from
Feb 1, 2014

Conversation

corysimmons
Copy link
Contributor

No description provided.

@japf
Copy link

japf commented Feb 1, 2014

Agreed with this suggestion even though node_modules folders could contain other stuff.

@pkozlowski-opensource
Copy link
Member

@corysimmons thnx for the pull request but I don't think this change is quite correct. We've removed the build folder and you are right that the README should be updated for this. But as commented by @japf node_modules has more than just grunt plugins.

@corysimmons
Copy link
Contributor Author

Sorry, I thought you were explicitly declaring things the clean client folder was containing. Updated.

pkozlowski-opensource added a commit that referenced this pull request Feb 1, 2014
@pkozlowski-opensource pkozlowski-opensource merged commit 9b11c05 into angular-app:master Feb 1, 2014
@pkozlowski-opensource
Copy link
Member

Thnx @corysimmons !

@corysimmons
Copy link
Contributor Author

De nada. Thanks for the app. Might get the book. =)

@corysimmons corysimmons deleted the patch-1 branch February 1, 2014 15:24
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.

3 participants