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

iss993: added DefaultProperties annotation [ready for review] #1259

Merged
merged 12 commits into from
Jul 6, 2016

Conversation

dmgcodevil
Copy link
Contributor

fix for #993

@dmgcodevil
Copy link
Contributor Author

PR in progress...

@mattrjacobs
Copy link
Contributor

Is this ready for review?

@dmgcodevil
Copy link
Contributor Author

Not yet, in progress. Will let you know when it's ready.
On 27 Jun 2016 9:18 a.m., "Matt Jacobs" notifications@github.com wrote:

Is this ready for review?


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1259 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ABWS8e2KLTAcGpfPdRVe0taecGM2jcsvks5qP820gaJpZM4I-vGS
.

@dmgcodevil
Copy link
Contributor Author

@mattrjacobs it's ready for review. BTW is it possible to use java 8 ? I'm still using java 6 for javanica.

@dmgcodevil dmgcodevil changed the title iss993: added DefaultProperties annotation [initial commit] iss993: added DefaultProperties annotation [ready for review] Jul 2, 2016
@spencergibb
Copy link
Contributor

If I recall, 1.x is maintaining JVM 6. If not I'd ask to wait for 2.0 to change that.

@dmgcodevil
Copy link
Contributor Author

It's ok. Will refactor it latter once 2.0 released. If PR is ok you can
merge it.
On 2 Jul 2016 9:37 p.m., "Spencer Gibb" notifications@github.com wrote:

If I recall, 1.x is maintaining JVM 6. If not I'd ask to wait for 2.0 to
change that.


You are receiving this because you authored the thread.
Reply to this email directly, view it on GitHub
#1259 (comment), or mute
the thread
https://github.com/notifications/unsubscribe/ABWS8cGf8C-z1wOTqnSB7Ca_VxOFurDvks5qRxJEgaJpZM4I-vGS
.

@mattrjacobs
Copy link
Contributor

Yes, let's hold off on Java8 until 2.0

@mattrjacobs
Copy link
Contributor

LGTM. Thanks @dmgcodevil ! I'll close #993. I'll get master in shape for a 1.5.4 release as well

@mattrjacobs mattrjacobs merged commit f918595 into Netflix:master Jul 6, 2016
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