Skip to content

Test the module in symfony 3.4 and php7.3+ #53

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 1 commit into from
Closed

Test the module in symfony 3.4 and php7.3+ #53

wants to merge 1 commit into from

Conversation

TavoNiievez
Copy link
Member

@TavoNiievez TavoNiievez commented Nov 23, 2020

This module is currently not being tested in Symfony 3.4 on PHP 7.3 and PHP 7.4 (and soon neither on php8) due to a error (you can check it in the CI Build).

Apparently this error originates from the test project code.

@TavoNiievez TavoNiievez added the help wanted Extra attention is needed label Nov 23, 2020
@Naktibalda
Copy link
Member

That error happens because that branch of symfony-demo uses version of Doctrine incompatible with PHP 7.3+

.travis.yml Outdated
@@ -16,10 +16,6 @@ env:
- VERSION=5 TEST_REPO='-b symfony5 https://github.com/Codeception/symfony-demo.git' SUITES=functional,unit
matrix:
exclude:
- php: 7.3
env: VERSION=3.4 TEST_REPO='--recurse-submodules https://github.com/Naktibalda/codeception-symfony-tests'
- php: 7.4
Copy link
Member

Choose a reason for hiding this comment

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

@eXorus deleted .travis.yml in other modules during migration to Github Actions.
I think that this file survived by accident and can be deleted.

Copy link
Member

Choose a reason for hiding this comment

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

Yes I confirmed.

Copy link
Member Author

@TavoNiievez TavoNiievez Nov 26, 2020

Choose a reason for hiding this comment

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

I removed Travis.
I am also closing this PR due to work started on #54, thanks for your feedback.

@TavoNiievez TavoNiievez mentioned this pull request Nov 26, 2020
@TavoNiievez TavoNiievez removed the help wanted Extra attention is needed label Nov 26, 2020
@TavoNiievez TavoNiievez deleted the test_sf_3.4 branch November 26, 2020 13:02
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.

3 participants