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

Remove jersey compatibility #95

Merged
merged 2 commits into from
Apr 7, 2016
Merged

Conversation

neykov
Copy link
Member

@neykov neykov commented Mar 31, 2016

My favourite kind of PRs - 0 additions and 9,808 deletions.

Now that 0.9.0 is branched we can remove the jersey compatibility.

@neykov
Copy link
Member Author

neykov commented Mar 31, 2016

Build failures fixed in #94. Once merged I expect some conflicts here, so will fix and trigger a rebuild.

@neykov neykov force-pushed the remove-jersey-compat branch 2 times, most recently from f3a138f to bb3cc84 Compare April 1, 2016 06:14
Test dependencies are not transitive, all need to be explicit.
@ahgittin
Copy link
Contributor

ahgittin commented Apr 1, 2016

delicious! however is the passwordless login mentioned in #92 fixed? not strictly a pre-req for this as we've already switched to cxf as the default but it is a high priority.

@neykov
Copy link
Member Author

neykov commented Apr 1, 2016

yes, completely fixed in #94

@sjcorbett
Copy link
Contributor

@ahgittin @neykov Are you ready for this to be merged?

@neykov
Copy link
Member Author

neykov commented Apr 7, 2016

Good to go. Thanks!

@asfgit asfgit merged commit 5838d16 into apache:master Apr 7, 2016
asfgit pushed a commit that referenced this pull request Apr 7, 2016
Remove jersey compatibility
@neykov neykov deleted the remove-jersey-compat branch April 7, 2016 11:17
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants