Skip to content

Unify how --format is handled by commands #19991

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.

Sign up for GitHub

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

Jump to bottom
Closed
fabpot opened this issue Jun 26, 2024 · 1 comment
Closed

Unify how --format is handled by commands #19991

fabpot opened this issue Jun 26, 2024 · 1 comment
Labels
hasPR A Pull Request has already been submitted for this issue.
Milestone
7.2

Comments

@fabpot
Copy link
Member

fabpot commented Jun 26, 2024

Q A
Feature PR symfony/symfony#57518
PR author(s) @fabpot
Merged in 7.2

We created this issue to not forget to document this new feature. We would really appreciate if you can help us with this task. If you are not sure how to do it, please ask us and we will help you.

To fix this issue, please create a PR against the 7.2 branch in the symfony-docs repository.

Thank you! 😃
@fabpot fabpot added this to the 7.2 milestone Jun 26, 2024
@javiereguiluz javiereguiluz added the hasPR A Pull Request has already been submitted for this issue. label Jun 26, 2024
@javiereguiluz
Copy link
Member

From the related code PR, this is the only change relevant for docs:

  • Deprecate the text format (used only twice) in favor of the more common txt one

In the docs we only mention the text format once and we use txt instead of text (see https://symfony.com/doc/current/frontend/asset_mapper.html#run-security-audits-on-your-dependencies) so there's nothing to update in the docs. Thanks!

@javiereguiluz javiereguiluz mentioned this issue Jun 26, 2024
Merged
javiereguiluz added a commit that referenced this issue Jun 27, 2024
@javiereguiluz
minor #19993 [Yaml] Minor fix when using the format option in the lin…
4e9cd70 
…t command (javiereguiluz)

This PR was merged into the 5.4 branch.

Discussion
----------

[Yaml] Minor fix when using the format option in the lint command

While reviewing #19991 I found this and I think it's an error.

Commits
-------

af46f19 [Yaml] Minor fix when using the format option in the lint command
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
hasPR A Pull Request has already been submitted for this issue.
Projects
None yet
Milestone
7.2
Development

No branches or pull requests
2 participants
@fabpot @javiereguiluz