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

Update the JRebel framework. #363

Merged
merged 1 commit into from
Dec 16, 2016
Merged

Update the JRebel framework. #363

merged 1 commit into from
Dec 16, 2016

Conversation

poolik
Copy link

@poolik poolik commented Dec 16, 2016

Use the 7.x release which has logging enabled by default, so no need to explictly set it. Also 'lib/jrebel.jar' doesn't exist anymore so removed it from the expectations.

Use the 7.x release which has logging enabled by default, so no need to explictly set it. Also 'lib/jrebel.jar' doesn't exist anymore so removed it from the expectations.
@cfdreddbot
Copy link

Hey poolik!

Thanks for submitting this pull request! I'm here to inform the recipients of the pull request that you and the commit authors have already signed the CLA.

@cgfrost
Copy link
Contributor

cgfrost commented Dec 16, 2016

LGTM and packages locally with JRebel 7.0.1 just fine.

@cgfrost cgfrost merged commit 8654879 into cloudfoundry:master Dec 16, 2016
@cgfrost
Copy link
Contributor

cgfrost commented Dec 16, 2016

Thank you for the contribution Tõnis.

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