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

PAYARA-3676 PAYARA-3665 Fix More Asadmin Prompt Issues #3870

Merged

Conversation

Pandrex247
Copy link
Member

Much more aggressive use of JLine to hopefully catch any remaining issues around asadmin prompts exiting after a single input character.

Fixes #3833

Copy link
Contributor

@jbee jbee left a comment

Choose a reason for hiding this comment

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

Common structure for the commands should be extracted to avoid duplication.

Copy link
Contributor

@Cousjava Cousjava left a comment

Choose a reason for hiding this comment

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

Copyright notices need updating; otherwise good

@Cousjava
Copy link
Contributor

Jenkins test please

1 similar comment
@Pandrex247
Copy link
Member Author

Jenkins test please

@mulderbaba mulderbaba added this to the 5.192 milestone Mar 30, 2019
@Pandrex247 Pandrex247 merged commit 71e4c2d into payara:master Apr 2, 2019
@Pandrex247 Pandrex247 deleted the PAYARA-3676-Asadmin-Prompt-Issues branch April 2, 2019 13:37
Pandrex247 added a commit to Pandrex247/Payara that referenced this pull request Jun 12, 2019
…ompt-Issues

PAYARA-3676 PAYARA-3665 Fix More Asadmin Prompt Issues
Pandrex247 added a commit to Pandrex247/Payara that referenced this pull request Jun 12, 2019
…a#618)"

This reverts pull request payara#618.

> * Merge pull request payara#3870 from Pandrex247/PAYARA-3676-Asadmin-Prompt-Issues
> 
>     PAYARA-3676 PAYARA-3665 Fix More Asadmin Prompt Issues
> 
> 
> * PAYARA-3665 Merge conflicts
> 
> ‌
> 
> Notable changes are in the second commit: String vs. char
AlanRoth pushed a commit to AlanRoth/Payara that referenced this pull request Aug 13, 2019
Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>
AlanRoth pushed a commit to AlanRoth/Payara that referenced this pull request Aug 13, 2019
Signed-off-by: Maxim Nesen <maxim.nesen@oracle.com>
Cousjava pushed a commit to Cousjava/Payara that referenced this pull request Aug 21, 2019
…ompt-Issues

PAYARA-3676 PAYARA-3665 Fix More Asadmin Prompt Issues
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.

5 participants