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

Allow for custom default command execution without arguments #109

Merged
merged 3 commits into from
Nov 20, 2024

Conversation

kodie
Copy link
Contributor

@kodie kodie commented Nov 18, 2024

Currently, if you set a custom default command and then try to run it without passing it any arguments, the help screen will be displayed just like it would have if we never set a custom default command.

This small PR makes it so that your default command can be ran without passing it any arguments.

@adhocore
Copy link
Owner

thank you for PR.

@dimtrovich could you check it too?

@dimtrovich
Copy link
Contributor

I've done a few tests and it's fine

@kodie please can you add a test (https://github.com/adhocore/php-cli/blob/main/tests/ApplicationTest.php)

@kodie
Copy link
Contributor Author

kodie commented Nov 19, 2024

@dimtrovich Done!

@adhocore
Copy link
Owner

thank you both 🎉

will release this tomorrow

@adhocore adhocore merged commit 0945f82 into adhocore:main Nov 20, 2024
6 checks passed
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