Skip to content
This repository was archived by the owner on May 28, 2018. It is now read-only.

Conversation

@walec51
Copy link

@walec51 walec51 commented Dec 3, 2014

No description provided.

@jerseyrobot
Copy link
Contributor

Can one of the admins verify this patch?

@AdamLindenthal
Copy link
Member

Jenkins, please test this patch.

@AdamLindenthal
Copy link
Member

Hi Adam (nice name, btw :)),
many thanks for your contribution.

Before we can review your pull request, we need you to sign Oracle Contributor Agreement.

More information about contributing to Jersey: https://jersey.java.net/scm.html#/Submitting_Patches_and_Contribute_Code

@AdamLindenthal
Copy link
Member

Verified manually, builds fine.

@walec51: The Valid annotation is missing the copyright header, could you please add it and recommit? You can take it from any class, just change the year, i.e. Copyright (c) 2014.

Thanks,
Adam

@walec51
Copy link
Author

walec51 commented Dec 8, 2014

ok, I've added the copyright and send the OCR to oracle-ca_us@oracle.com

@japod
Copy link
Member

japod commented Dec 10, 2014

Jenkins, please test this patch.

@walec51
Copy link
Author

walec51 commented Dec 18, 2014

whats the status of this pull request ?
the failed test seams to have nothing to do with my change
besides I cannot run it my self becuse this dependancy org.glassfish.jersey.media:jersey-media-sse is not available according to maven

mpotociar added a commit that referenced this pull request Dec 18, 2014
JERSEY-2721 Fixed proxy client handling of entity method parameter annotations.
@mpotociar mpotociar merged commit 3909277 into javaee:master Dec 18, 2014
@mpotociar
Copy link
Collaborator

Thank you for your contribution.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants