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

[Console] allow the answer to not be trimmed #11603

Merged

Conversation

Simperfit
Copy link
Contributor

This documents the new flag in the Question class to allow the answer to not be trimmed

@OskarStark OskarStark added the Waiting Code Merge Docs for features pending to be merged label May 27, 2019
@OskarStark OskarStark added this to the 4.4 milestone May 27, 2019
@Simperfit Simperfit changed the base branch from master to 4.4 June 5, 2019 21:00
fabpot added a commit to symfony/symfony that referenced this pull request Jul 8, 2019
…(Simperfit)

This PR was merged into the 4.4 branch.

Discussion
----------

[Console] allow answer to be trimmed by adding a flag

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | none   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#11603 <!-- required for new features -->

<!--
Replace this notice by a short README for your feature/bugfix. This will help people
understand your PR and can be used as a start for the documentation.

Additionally (see https://symfony.com/roadmap):
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->

According to #23210 (comment) we add a new flag in the `Question` class to be able to not trim the answer.

Commits
-------

8f182d8 [Console] allow answer to be trimmed by adding a flag
symfony-splitter pushed a commit to symfony/console that referenced this pull request Jul 8, 2019
…(Simperfit)

This PR was merged into the 4.4 branch.

Discussion
----------

[Console] allow answer to be trimmed by adding a flag

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | no
| New feature?  | yes <!-- please update src/**/CHANGELOG.md files -->
| BC breaks?    | no     <!-- see https://symfony.com/bc -->
| Deprecations? | no <!-- please update UPGRADE-*.md and src/**/CHANGELOG.md files -->
| Tests pass?   | yes    <!-- please add some, will be required by reviewers -->
| Fixed tickets | none   <!-- #-prefixed issue number(s), if any -->
| License       | MIT
| Doc PR        | symfony/symfony-docs#11603 <!-- required for new features -->

<!--
Replace this notice by a short README for your feature/bugfix. This will help people
understand your PR and can be used as a start for the documentation.

Additionally (see https://symfony.com/roadmap):
 - Bug fixes must be submitted against the lowest maintained branch where they apply
   (lowest branches are regularly merged to upper ones so they get the fixes too).
 - Features and deprecations must be submitted against the master branch.
-->

According to symfony/symfony#23210 (comment) we add a new flag in the `Question` class to be able to not trim the answer.

Commits
-------

8f182d811e [Console] allow answer to be trimmed by adding a flag
@OskarStark OskarStark removed the Waiting Code Merge Docs for features pending to be merged label Aug 8, 2019
Copy link
Contributor

@OskarStark OskarStark left a comment

Choose a reason for hiding this comment

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

some minor comments

@Simperfit Simperfit force-pushed the feature/allow-answer-to-be-trimmed branch from 7c2e9cf to ce28de7 Compare August 8, 2019 12:28
@Simperfit
Copy link
Contributor Author

updated @OskarStark

@OskarStark OskarStark force-pushed the feature/allow-answer-to-be-trimmed branch from 1f98bfc to dcb0eaa Compare August 9, 2019 11:18
@OskarStark
Copy link
Contributor

Thank you Hamza.

@OskarStark OskarStark merged commit dcb0eaa into symfony:4.4 Aug 9, 2019
OskarStark added a commit that referenced this pull request Aug 9, 2019
This PR was squashed before being merged into the 4.4 branch (closes #11603).

Discussion
----------

[Console] allow the answer to not be trimmed

<!--

If your pull request fixes a BUG, use the oldest maintained branch that contains
the bug (see https://symfony.com/roadmap for the list of maintained branches).

If your pull request documents a NEW FEATURE, use the same Symfony branch where
the feature was introduced (and `master` for features of unreleased versions).

-->

This documents the new flag in the Question class to allow the answer to not be trimmed

Commits
-------

dcb0eaa [Console] allow the answer to not be trimmed
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.

4 participants