Skip to content
This repository has been archived by the owner on Sep 26, 2019. It is now read-only.

Fix NC-1707 pantheon CLI default values infinite loop #266

Conversation

NicolasMassart
Copy link
Contributor

PR description

The issue of infinite loop was due to an incorrect way to iterate over lines to split them to the right length while displaying descriptions with placeholders like ${DEFAULT-VALUE}

Picocli latest release 3.8.0 fixes the issue, so here is the update and default values are back in descriptions.

Waiting for PR #265 to be merged before merging this one as there's probably some conflicts.

Fixed Issue(s)

fixes NC-1707

@NicolasMassart NicolasMassart added the work in progress Work on this pull request is ongoing label Nov 15, 2018
@NicolasMassart NicolasMassart self-assigned this Nov 15, 2018
@NicolasMassart NicolasMassart removed the work in progress Work on this pull request is ongoing label Nov 16, 2018
Copy link
Contributor

@ajsutton ajsutton left a comment

Choose a reason for hiding this comment

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

LGTM.

@ajsutton ajsutton added the bug Something isn't working label Nov 19, 2018
@NicolasMassart NicolasMassart merged commit c12e1f5 into PegaSysEng:master Nov 19, 2018
@NicolasMassart NicolasMassart deleted the fix/NC-1707_picocli_infinite_loop branch December 7, 2018 08:10
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants