Thanks to visit codestin.com
Credit goes to github.com

Skip to content

[Console] Add note about URL cutting of default output wrapper #16476

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

Merged

Conversation

fchris82
Copy link
Contributor

@fchris82 fchris82 commented Feb 4, 2022

Add note about the new output wrapper function. Related MR: symfony/symfony#45318

@carsonbot carsonbot added this to the 6.1 milestone Feb 4, 2022
@javiereguiluz javiereguiluz added the Waiting Code Merge Docs for features pending to be merged label Mar 11, 2022
@carsonbot carsonbot modified the milestones: 6.1, next Mar 11, 2022
fabpot added a commit to symfony/symfony that referenced this pull request Jul 26, 2022
… (fchris82, GromNaN)

This PR was merged into the 6.2 branch.

Discussion
----------

[Console] Don't cut Urls wrapped in SymfonyStyle block

| Q             | A
| ------------- | ---
| Branch?       | 6.2
| Bug fix?      | no
| New feature?  | yes
| Deprecations? | no
| Tickets       | #30920
| License       | MIT
| Doc PR        | symfony/symfony-docs#16476

Continuation of #45318 by fchris82

Commits
-------

b317f06 Remove OutputWrapperInterface
2bc2571 Console output formatter improvement
@javiereguiluz javiereguiluz removed the Waiting Code Merge Docs for features pending to be merged label Jul 29, 2022
@javiereguiluz javiereguiluz changed the base branch from 6.1 to 6.2 September 22, 2022 07:42
@javiereguiluz javiereguiluz force-pushed the console-output-formatter-improvement branch from d7ea43d to 0bfa8ac Compare September 22, 2022 07:43
@javiereguiluz javiereguiluz merged commit 4a3d165 into symfony:6.2 Sep 22, 2022
@javiereguiluz
Copy link
Member

@fchris82 thanks a lot for this contribution! Please note that while merging, we relocated and expanded your text to create a whole new section in the article. See adfe2d2

In any case, thanks a lot and congrats on your first Symfony Docs contribution 🎉

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.

3 participants