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

Conversation

@marcuslinke
Copy link

This fixes https://java.net/jira/browse/JERSEY-2852

Please review and merge. Thanks!

@mgajdos
Copy link
Contributor

mgajdos commented May 15, 2015

Hi @marcuslinke,
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…

Also provide some tests that would cover new code and add copyright header to the new file. Thanks.

Regards,
Michal

@AdamLindenthal
Copy link
Member

Hi Marcus,
have you noticed Michal's reponse? Any follow-up? I mean - are you still planning to continue the work on this?

Thanks,
Adam

@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 Marcus, will you sign and send the OCA? Or have you already done so?
Please, let us know. Also, we will need you to create a unit test for your change.

Thanks,
Adam

@AdamLindenthal
Copy link
Member

Hi Marcus,

I am sorry, but since you didn't respond since may and didn't sign the OCA, I am closing the PR.
Once you have more time to finish this, feel free to re-try.

Regards,
Adam

@KostyaSha
Copy link

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