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

some more basic cleanup and fixing #29

Merged
merged 5 commits into from Nov 22, 2015
Merged

Conversation

breskeby
Copy link
Contributor

This fixes an classloading issue with using gradle internal LazyEvaluationFileCollection

Furthermore it contains some more cleanup:

  • more integ test coverage for referencing java from frege
  • initial integ test for FregeCompile
  • remove @optional annotations from FregeCompile properties; @optional was missused as it does not indicate preconfigured options, but options that can be null
  • move integ tests into correct packages
  • minor renaming on properties in FregeCompile
  • fix for handling fregepath

- get packages for integ tests right
- introduce common AbstractFregeIntegrationSpec
- first stab of unit test coverage for FregeCompile
- minor cleanup on FregeCompile
- configure fregePath as part as part of base plugin convention
Dierk pushed a commit that referenced this pull request Nov 22, 2015
some more basic cleanup and fixing
@Dierk Dierk merged commit e2575d2 into Frege:master Nov 22, 2015
@Dierk
Copy link
Member

Dierk commented Nov 22, 2015

Awesome! Thanks a lot for your extra-time work!

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