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

Disable platform checks #38818

Merged
merged 1 commit into from
Oct 26, 2020
Merged

Conversation

nicolas-grekas
Copy link
Member

Q A
Branch? 3.4
Bug fix? no
New feature? no
Deprecations? no
Tickets Fix #38803
License MIT
Doc PR -

Let's see

@nicolas-grekas nicolas-grekas merged commit 777658c into symfony:3.4 Oct 26, 2020
@jderusse
Copy link
Member

@stof
Copy link
Member

stof commented Oct 26, 2020

weird that this happens only for one specific sub-test. Could it be that this one does special things for its composer usage and does not use that config file ?

@jderusse
Copy link
Member

Other version are compiled with mbstring (see php -i output for php 7.4 for instance)

@jderusse
Copy link
Member

same for 7.2 on branch 5.x https://travis-ci.org/github/symfony/symfony/builds/738951096

@jderusse
Copy link
Member

fixed by 255f1e3

@nicolas-grekas nicolas-grekas deleted the pub-no-checks branch October 26, 2020 17:04
nicolas-grekas added a commit that referenced this pull request Oct 27, 2020
This PR was merged into the 3.4 branch.

Discussion
----------

Display php info for extra versions in travis

| Q             | A
| ------------- | ---
| Branch?       | 3.4
| Bug fix?      | no
| New feature?  | no
| Deprecations? | no
| Tickets       | -
| License       | MIT
| Doc PR        | -

Display the PHP info of all tested versions when the test-suite use several versions of PHP

Helps to troubleshoot  #38818 (comment)

Commits
-------

25a7333 Display php info for extra versions in travis
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