10000 WIP: Add PHP 8.1 job to Travis by derrabus · Pull Request #40532 · symfony/symfony · GitHub
[go: up one dir, main page]

Skip to content

WIP: Add PHP 8.1 job to Travis #40532

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

Conversation

derrabus
Copy link
Member
Q A
Branch? 4.4
Bug fix? no
New feature? no
Deprecations? no
Tickets N/A
License MIT
Doc PR N/A

We should start testing on PHP 8.1 soon-ish.

@carsonbot carsonbot added this to the 4.4 milestone Mar 21, 2021
@derrabus derrabus force-pushed the improvement/travis-nightly branch 2 times, most recently from d2bf821 to 39247f8 Compare March 21, 2021 21:17
@derrabus
Copy link
Member Author

Apparently, Travis runs PHP 8.0 as php: nightly. 😕

@jrushlow
Copy link
Contributor

Shouldn't we hold off until the beta's start rolling out at the end of July? Maybe this would allow us to implement this in GH Actions instead of Travis if that is the overall direction we are headed with CI.

@derrabus
Copy link
Member Author

@jrushlow I'd like to have a CI job for 8.1 before the beta, in order to monitor Symfony's compatibility. That job would be allowed to fail, of course.

  • We can provide early feedback to the php team.
  • We would enable libraries and apps that depend on Symfony to test early.

@derrabus derrabus force-pushed the improvement/travis-nightly branch from 39247f8 to 28b030c Compare April 2, 2021 15:34
@derrabus derrabus force-pushed the improvement/travis-nightly branch from 28b030c to b1f4eb1 Compare May 2, 2021 23:59
Nyholm added a commit that referenced this pull request May 15, 2021
… Nyholm)

This PR was merged into the 4.4 branch.

Discussion
----------

Add a GitHub Action for tests against PHP 8.1

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

I'd like to monitor our compatibility with the upcoming PHP 8.1 release. Because I failed miserably at convincing Travis to run our tests with PHP 8.1 (see #40532), I tried to do the same with GitHub Actions.

Commits
-------

6a328ae Update .github/workflows/tests.yml
6f1c4b8 PHP 8.1 must not cause the build to fail (yet)
b0b4510 Add _run_tests bash function
de4f017 Add a GitHub Action for tests against PHP 8.1
@derrabus derrabus deleted the improvement/travis-nightly branch June 29, 2021 13:11
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
0