Skip to content

Update PHPUnit and PHPCS, require 8.2+#39

Merged
Firehed merged 15 commits into
masterfrom
update-dependencies
May 9, 2026
Merged

Update PHPUnit and PHPCS, require 8.2+#39
Firehed merged 15 commits into
masterfrom
update-dependencies

Conversation

@Firehed
Copy link
Copy Markdown
Owner

@Firehed Firehed commented May 9, 2026 •
edited
Loading

  • Updates to current PHPUnit versions
  • Updates to current PHPCS
  • PHPStan ignored, will deal with it in a separate PR
  • Bumped min PHP to 8.2, oldest currently supported version
  • Update test syntax for newer PHPUnit

Firehed and others added 15 commits May 9, 2026 15:35
@Firehed
Update all the things
ea132b1
@Firehed
Update phpunit config
7e2fccd
@Firehed
Update gitignore
ab6e5c7
@Firehed @claude
Convert PHPUnit annotations to attributes
ecfe020 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed @claude
Replace ->will($this->returnValue()) with ->willReturn()
2893985 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed @claude
Replace ->will($this->throwException()) with ->willThrowException()
97b88a4 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed @claude
Add missing data provider argument to test methods
c5e47dd 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed
Add return types
9268a53
@Firehed @claude
Replace setMethods() with onlyMethods()
943688f 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed @claude
Replace getMockForAbstractClass() with createStub()/getMock()
da35e77 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed @claude
Use mocks-as-stubs with AllowMockObjectsWithoutExpectations
2f00a2f 
Co-Authored-By: Claude Opus 4.5 <noreply@anthropic.com>
@Firehed
Leave PHPStan alone for now
cfa1dcf
@Firehed
phpcs
9dc65a7
@Firehed
update actions
ca55a2f
@Firehed
Lock phpstan to 8.2 for now
74db853
@Firehed Firehed changed the title Update dependencies Update PHPUnit and PHPCS, require 8.2+ May 9, 2026
@Firehed
Copy link
Copy Markdown
Owner Author

Firehed commented May 9, 2026

As noted in PR body, ignoring phpstan for now.

@Firehed Firehed marked this pull request as ready for review May 9, 2026 23:06
@Firehed Firehed merged commit e56d8e3 into master May 9, 2026
3 of 4 checks passed
@Firehed Firehed deleted the update-dependencies branch May 9, 2026 23:08
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

1 participant

@Firehed