Skip to content

Display title if exists if a required argument is missing#89

Open
l-vo wants to merge 1 commit into
nategood:masterfrom
l-vo:add_title_required_error
Open

Display title if exists if a required argument is missing#89
l-vo wants to merge 1 commit into
nategood:masterfrom
l-vo:add_title_required_error

Conversation

@l-vo

@l-vo l-vo commented May 29, 2018

Copy link
Copy Markdown

In the help screen, the title can be displayed instead of "arg 0". Add the same behavior on the error message when a required argument is missing.

@NeoVance

Copy link
Copy Markdown
Contributor

@l-vo Please merge latest dev, and rerun checks.

@l-vo l-vo force-pushed the add_title_required_error branch from 9f07004 to e0d0b03 Compare July 16, 2018 11:10
@l-vo

l-vo commented Jul 16, 2018

Copy link
Copy Markdown
Author

@NeoVance Done :)

@tolidano

Copy link
Copy Markdown

I tried to do a similar thing because I didn't read carefully enough the list of open PRs: #98

@l-vo

l-vo commented May 14, 2019

Copy link
Copy Markdown
Author

@tolidano Sadly I'm not sure that the project is still maintained anyway. There are some alternatives like Symfony console or Laravel zero (although they are a bit heavier). @nategood could you confirm the state of the project please ?

@tolidano

tolidano commented May 15, 2019

Copy link
Copy Markdown

if @nategood wants to abandon, I offer to take it over, I think it's far and away more useful than pulling in Symfony or Laravel components.
Feel free to publish a new version to packagist to redirect to tolidano/commando, and I'll set up the rest. I'll even go through all the open PRs and the issues. I'll publish a final version then upgrade the whole thing to PHP 7.2 and publish a 7.2+ version.

@tolidano

Copy link
Copy Markdown

I went ahead and forked and rewrote this for PHP 7.2 - https://packagist.org/packages/tolidano/commandox

@tolidano

Copy link
Copy Markdown

I created a PHP 5.4-7.1 version of this and then a PHP 7.2 version: https://github.com/tolidano/commandox/releases

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.

3 participants