Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

#

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Jump to bottom

CommandRegistry interface: removed execute() method #543

Merged
merged 1 commit into from
Jun 6, 2020
Merged

CommandRegistry interface: removed execute() method #543

merged 1 commit into from
Jun 6, 2020

Conversation

mattirn
Copy link
Collaborator

@mattirn mattirn commented Jun 6, 2020 •
edited
Loading

CommandRegistry interface has been removed two methods:

  1. execute(CommandSession session, String command, String[] args)
    You should implement instead invoke(CommandSession session, String command, Object... args)

  2. CmdDesc commandDescription(String command)
    You should implement instead CmdDesc commandDescription(List<String> args)

@mattirn mattirn mentioned this pull request Jun 6, 2020
Closed
@mattirn mattirn added this to the 3.15.1 milestone Jun 6, 2020
@mattirn mattirn merged commit 497f10c into jline:master Jun 6, 2020
mattirn added a commit that referenced this pull request Jun 6, 2020
@mattirn
CommandRegistry interface: removed method commandDescription(command),
de55886 
#543
@mattirn mattirn deleted the commandRegistry-improvment branch June 9, 2020 16:35
@mattirn mattirn mentioned this pull request Jul 11, 2020
Closed
# for free to join this conversation on GitHub. Already have an account? # to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
3.16.0
Development

Successfully merging this pull request may close these issues.
1 participant
@mattirn