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

Drop support for PHP < 8.1, upgrade to PHPUnit 10 #641

Merged
merged 7 commits into from
Nov 21, 2023

Conversation

Slamdunk
Copy link
Contributor

@Slamdunk Slamdunk commented Nov 21, 2023

Blocked by #640

  • Drop support for PHP < 8.1
  • Always run on ubuntu-latest; remove it from matrix to get a cleaner GH status name
  • Upgrade to PHPUnit 10

Build green on my fork:
https://github.com/Slamdunk/browscap-php/actions/runs/6941992974

composer.json Outdated Show resolved Hide resolved
@Slamdunk Slamdunk marked this pull request as ready for review November 21, 2023 13:57
@Slamdunk
Copy link
Contributor Author

Hi, since #640 has been merged, you should as well update the list of Required steps in the page https://github.com/browscap/browscap-php/settings/branches

@asgrim
Copy link
Member

asgrim commented Nov 21, 2023

Hi, since #640 has been merged, you should as well update the list of Required steps in the page https://github.com/browscap/browscap-php/settings/branches

This was because you dropped the OS from the matrix in this PR, not relevant to #640 - anyway I've updated it to work for the new job names. Not sure why I can't just ask for "green builds" and have to specify each job but hey. We can fix that later :)

@asgrim asgrim self-assigned this Nov 21, 2023
@asgrim asgrim added this to the 7.3.0 milestone Nov 21, 2023
@asgrim asgrim merged commit d5c7dcc into browscap:7.3.x Nov 21, 2023
30 checks passed
@Slamdunk Slamdunk deleted the drop_old_php branch November 21, 2023 14:55
@Slamdunk
Copy link
Contributor Author

Yeah sorry, wrong link, I meant after 85485dc

Thank you

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.

3 participants