8000 Handle false for shortcut by driesvints · Pull Request #53702 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

Conversation

@driesvints
8000 Copy link
Contributor
Q A
Branch? 5.4
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #53516
License MIT

@carsonbot
Copy link

Hey!

To help keep things organized, we don't allow "Draft" pull requests. Could you please click the "ready for review" button or close this PR and open a new one when you are done?

Note that a pull request does not have to be "perfect" or "ready for merge" when you first open it. We just want it to be ready for a first review.

Cheers!

Carsonbot

@driesvints driesvints closed this Jan 31, 2024
@driesvints driesvints deleted the patch-1 branch January 31, 2024 13:51
nicolas-grekas added a commit that referenced this pull request Feb 1, 2024
…nsilicon)

This PR was squashed before being merged into the 5.4 branch.

Discussion
----------

[Console] Allow false as a $shortcut in InputOption

| Q             | A
| ------------- | ---
| Branch?       | 5.4
| Bug fix?      | yes
| New feature?  | no
| Deprecations? | no
| Issues        | Fix #53702
| License       | MIT

This change is needed for laravel 8.x installation to allow running kernel and module based architecture. Right now our deployments are suddenly stopped since morning all saying "An option shortcut cannot be empty" and root cause for this is $shortcut is coming as false from laravel modules feature library.

Commits
-------

2025baa [Console] Allow false as a $shortcut in InputOption
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.

2 participants

0