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

Add guava flavour (needs test!) #47

Merged
merged 4 commits into from
Aug 9, 2016
Merged

Add guava flavour (needs test!) #47

merged 4 commits into from
Aug 9, 2016

Conversation

h908714124
Copy link
Contributor

I would have loved to add a test but wasn't sure where to put it

@jbgi
Copy link
Member

jbgi commented Aug 9, 2016

Looks good to me! thanks!
For now the tests is mainly the example module: you can add a new data type that use the Guava flavor in there.

@jbgi
Copy link
Member

jbgi commented Aug 9, 2016

Also it looks like your commit is associated with another github account of yours (@anon789) are you ok with that?

@h908714124
Copy link
Contributor Author

No idea what that other account is, I don't care.

@h908714124
Copy link
Contributor Author

Hang on I'll add a test

@h908714124
Copy link
Contributor Author

h908714124 commented Aug 9, 2016

jenkins problem

@jbgi
Copy link
Member

jbgi commented Aug 9, 2016

@h908714124 ah yeah forgot about that: it just need a bsd header. you can add it via "./gradlew licenseFormatTest".
I should probably remove this check from the default build...

@h908714124
Copy link
Contributor Author

didn't help

@jbgi
Copy link
Member

jbgi commented Aug 9, 2016

ok... sorry forgot about the other file in src/main... that is taken care of by "./gradlew licenseFormatMain".

@jbgi jbgi merged commit eb067d8 into derive4j:master Aug 9, 2016
@jbgi
Copy link
Member

jbgi commented Aug 9, 2016

yeah!! thanks!

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

3 participants