Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content

Gradle#131

Closed
Haringat wants to merge 5 commits intostleary:masterfrom
Haringat:gradle
Closed

Gradle#131
Haringat wants to merge 5 commits intostleary:masterfrom
Haringat:gradle

Conversation

@Haringat
Copy link

@Haringat Haringat commented Jun 5, 2015

As mentioned in the other PR I added the automated build system gradle.
The other changes are you see are either directly related to that (2nd moving of the source files) or the changes I made in the other PR.

Haringat added 5 commits June 4, 2015 05:21
See the documentation for details. Also changed the documentation accordingly.
this made it necessary to move the source files (again)
also added a .gitignore
@Haringat Haringat mentioned this pull request Jun 5, 2015
@johnjaylward
Copy link
Contributor

I can't see a pull request this large being approved. You should break out your feature changes into separate pull requests so they are easier to merge and test individually.

Also, all previous build tool implementations have been declined. For example, #46, #35, #21. Unless you can show a good reason for implementing it on such a simple project, I can't see it happening.

@stleary
Copy link
Owner

stleary commented Jun 5, 2015

Thanks for the offer, but there are no plans at this time for this project to support any build automation.

@stleary stleary closed this Jun 5, 2015
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