Skip to content

cleanup#10

Merged
bmuschko merged 6 commits intobmuschko:masterfrom
burtbeckwith:master
Nov 12, 2012
Merged

cleanup#10
bmuschko merged 6 commits intobmuschko:masterfrom
burtbeckwith:master

Conversation

@burtbeckwith
Copy link
Copy Markdown

No description provided.

@bmuschko
Copy link
Copy Markdown
Owner

Good stuff! Thanks for the pull request.

I am personally not a fan of checking in IDE project files into VCS as they can easily be generated in most cases. Was there a specific reason to why you wanted to add the STS project files?

@burtbeckwith
Copy link
Copy Markdown
Author

They can be generated, but the ones I committed are updated from the ones that Grails generates - they work with the latest version of GGTS (it currently has to fix the ones Grails generates) and unused folders are excuded. I also change the GGTS class directory to target/eclipseclasses so everything that's generated is in one place. But of course feel free to revert that commit - I intentionally split the changes into separate commits so you can choose which to incorporate.

@bmuschko bmuschko merged commit 5c1bdf0 into bmuschko:master Nov 12, 2012
@ghost ghost assigned bmuschko Nov 12, 2012
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