Skip to content

use correct ParseOptions object#709

Merged
havocp merged 1 commit intolightbend:masterfrom
bbaldino:fix_parse_replacement
Sep 24, 2020
Merged

use correct ParseOptions object#709
havocp merged 1 commit intolightbend:masterfrom
bbaldino:fix_parse_replacement

Conversation

@bbaldino
Copy link
Contributor

#708 has an issue with calling ensureClassLoader but not using the result, this PR fixes that.

Note: I'm not sure where the formatting fixes came from, could that be from sbt test? I ran that locally. I can get rid of them if needed.

Copy link
Contributor

@havocp havocp left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks, sorry I missed that in round 1. I think cleaning up trailing whitespace is good, not sure where it came from either, but let's go for it.

@havocp havocp merged commit 82df5a6 into lightbend:master Sep 24, 2020
@bbaldino
Copy link
Contributor Author

Np, my bad for not following the other uses. Do we need to make sure this PR is mentioned in #702 for the release?

@havocp
Copy link
Contributor

havocp commented Sep 24, 2020

I added a comment over there but yeah it should be in NEWS.md ideally

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.

2 participants