Skip to content

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

Merged
asgrim merged 7 commits intobrowscap:7.3.xfrom
Slamdunk:drop_old_php
Nov 21, 2023
Merged

Drop support for PHP < 8.1, upgrade to PHPUnit 10#641
asgrim merged 7 commits intobrowscap:7.3.xfrom
Slamdunk:drop_old_php

Conversation

@Slamdunk
Copy link
Copy Markdown
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

@Slamdunk Slamdunk marked this pull request as ready for review November 21, 2023 13:57
@Slamdunk
Copy link
Copy Markdown
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
Copy Markdown
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
@Slamdunk Slamdunk deleted the drop_old_php branch November 21, 2023 14:55
@Slamdunk
Copy link
Copy Markdown
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