Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

[FrameworkBundle] Update console fixtures after #22217 #22278

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

Closed
wants to merge 2 commits into from
Closed

[FrameworkBundle] Update console fixtures after #22217 #22278

wants to merge 2 commits into from

Conversation

ro0NL
Copy link
Contributor

@ro0NL ro0NL commented Apr 4, 2017

Q A
Branch? 2.8
Bug fix? yes (regression from #22217)
New feature? no
BC breaks? no
Deprecations? no
Tests pass? yes
Fixed tickets #22217 (comment)
License MIT
Doc PR symfony/symfony-docs#...

cc @fabpot

Sorry for the inconvenience :)

@ro0NL
Copy link
Contributor Author

ro0NL commented Apr 4, 2017

Travis is hanging?

@fabpot
Copy link
Member

fabpot commented Apr 4, 2017

@ro0NL No, it's just that there is a lot of activity, and the queue is quite large right now :)

@ro0NL ro0NL changed the base branch from master to 2.8 April 4, 2017 15:13
Copy link
Member

@chalasr chalasr left a comment

Choose a reason for hiding this comment

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

👍

@ro0NL
Copy link
Contributor Author

ro0NL commented Apr 4, 2017

Monitoring travis.. and PHP7 still fails. What a day...

edit: wait.. that could actually be a deps issue.. ill have a look at composer file.

@ro0NL
Copy link
Contributor Author

ro0NL commented Apr 4, 2017

Not sure what needs to be done here;

# frameworkbundle composer.json
"symfony/console": "~2.8.8|~3.0.8",
# travis, PHP7, deps=high
- Installing symfony/console (3.0.x-dev 926061e): Loading from cache
# travis, PHP7.1, deps=low
- Installing symfony/console (v2.8.8): Downloading (100%)

@chalasr
Copy link
Member

chalasr commented Apr 4, 2017

@ro0NL Not sure but I'd say ~2.8.19|~3.3

@fabpot
Copy link
Member

fabpot commented Apr 4, 2017

No, in 2.8, we cannot depend on anything else but 3.0. I think we dropped 3.0 in some cases though, so ~2.8.19 should be the way to go I suppose. Actually, that's a bad idea.

@ro0NL
Copy link
Contributor Author

ro0NL commented Apr 4, 2017

🎉 thx @chalasr for helping out and pointing out these issues.

@@ -38,7 +38,7 @@
},
"require-dev": {
"symfony/browser-kit": "~2.4|~3.0.0",
"symfony/console": "~2.8.8|~3.0.8",
"symfony/console": "~2.8.19|~3.3",
Copy link
Member

@nicolas-grekas nicolas-grekas Apr 4, 2017

Choose a reason for hiding this comment

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

~2.8.19|~3.2.7 ?

@nicolas-grekas
Copy link
Member

Thank you @ro0NL.

nicolas-grekas added a commit that referenced this pull request Apr 4, 2017
…ro0NL)

This PR was squashed before being merged into the 2.8 branch (closes #22278).

Discussion
----------

[FrameworkBundle] Update console fixtures after #22217

| Q             | A
| ------------- | ---
| Branch?       | 2.8
| Bug fix?      | yes (regression from #22217)
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #22217 (comment)
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!--highly recommended for new features-->

cc @fabpot

Sorry for the inconvenience :)

Commits
-------

6e1cee6 [FrameworkBundle] Update console fixtures after #22217
@nicolas-grekas
Copy link
Member

merged, can you please look now at master which is still failing?

@ro0NL ro0NL deleted the console/table-cell branch April 4, 2017 21:12
fabpot added a commit that referenced this pull request Apr 4, 2017
This PR was merged into the 3.3-dev branch.

Discussion
----------

[FrameworkBundle] Fix console fixtures

| Q             | A
| ------------- | ---
| Branch?       | master
| Bug fix?      | yes
| New feature?  | no
| BC breaks?    | no
| Deprecations? | no
| Tests pass?   | yes
| Fixed tickets | #22278 (comment)
| License       | MIT
| Doc PR        | symfony/symfony-docs#... <!--highly recommended for new features-->

cc @nicolas-grekas

Commits
-------

a7bd944 [FrameworkBundle] Fix console fixtures
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.

4 participants
Morty Proxy This is a proxified and sanitized view of the page, visit original site.