Skip to content

Conversation

@SiebeVE
Copy link
Contributor

@SiebeVE SiebeVE commented Dec 12, 2025

Q A
Branch? 7.3
Bug fix? yes
New feature? no
Deprecations? no
Issues Fix #62763
License MIT

@SiebeVE SiebeVE requested a review from chalasr as a code owner December 12, 2025 15:50
@carsonbot carsonbot added this to the 7.3 milestone Dec 12, 2025
@carsonbot carsonbot changed the title Bug #62763 [Console] 7.3 [Console] Bug #62763 7.3 Dec 12, 2025
@SiebeVE SiebeVE changed the title [Console] Bug #62763 7.3 [Console] Bug #62763 Description of console command not escaped Dec 12, 2025
@GromNaN GromNaN changed the title [Console] Bug #62763 Description of console command not escaped [Console] Escape % in description of console commands Dec 12, 2025
@GromNaN
Copy link
Member

GromNaN commented Dec 12, 2025

Hello @SiebeVE,

Thank you for the issue and the PR. You may have found the fix right after creating the issue, however, creating an issue is not required to open a PR.

Could you rebase your branch to remove the merge commit?

… command so the container can be build without wanting to replace the %not-a-param% to parameters
@SiebeVE
Copy link
Contributor Author

SiebeVE commented Dec 12, 2025

Aha, cool, won't create an issue then in the future :-).

I've done a rebase, so the merge commit should be gone now.

Copy link
Member

@yceruto yceruto left a comment

Choose a reason for hiding this comment

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

Thanks!

@nicolas-grekas
Copy link
Member

Thank you @SiebeVE.

@nicolas-grekas nicolas-grekas merged commit 0dd7d93 into symfony:7.3 Dec 13, 2025
11 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants