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

Skip to content

Configure services additionally required by the master branch #36704

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

jakzal
Copy link
Contributor

@jakzal jakzal commented May 5, 2020

Q A
Branch? master
Bug fix? no
New feature? no
Deprecations? no
Tickets Re #36647
License MIT
Doc PR -

Additionally run tests for:

Before: Tests: 1893, Assertions: 5178, Skipped: 105.
After: Tests: 2042, Assertions: 5407, Skipped: 120.

@jakzal jakzal marked this pull request as ready for review May 5, 2020 11:11
@jakzal jakzal requested review from jderusse and sroze as code owners May 5, 2020 11:11
@jakzal jakzal force-pushed the tests-on-github-actions-master branch from e9d628c to f0472a7 Compare May 5, 2020 11:12
@nicolas-grekas nicolas-grekas added this to the 5.1 milestone May 5, 2020
@jakzal jakzal force-pushed the tests-on-github-actions-master branch from b547ace to 2aca92b Compare May 5, 2020 14:30
@nicolas-grekas
Copy link
Member

I fixed Travis on master/5.0 and created a small conflict with your PR. Can you please rebase?

@nicolas-grekas
Copy link
Member

nicolas-grekas commented May 5, 2020

Removing the 2nd commit is enough.
I would have done it for you if github allowed pushing on your fork.
I'm sad they changed the default value for this checkbox :'(

@jakzal jakzal force-pushed the tests-on-github-actions-master branch from 2aca92b to dc7ac57 Compare May 5, 2020 19:37
@jakzal
Copy link
Contributor Author

jakzal commented May 5, 2020

Removed the second commit and rebased 👍

@nicolas-grekas
Copy link
Member

Thank you @jakzal.

@nicolas-grekas nicolas-grekas merged commit 78df4ca into symfony:master May 5, 2020
@jakzal jakzal deleted the tests-on-github-actions-master branch May 5, 2020 19:47
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