Skip to content

Conversation

sidz
Copy link
Collaborator

@sidz sidz commented Jul 23, 2025

This PR updates phpstan to the 2.1 and resolve some of the issues.

@sidz sidz force-pushed the update-phpstan branch 6 times, most recently from d467709 to 9f81f52 Compare July 23, 2025 20:54
@sidz sidz force-pushed the update-phpstan branch from 9f81f52 to 168fe0e Compare July 23, 2025 21:04
@@ -18,9 +18,9 @@
/**
* @internal
*/
#[Group('unit')]
Copy link
Owner

Choose a reason for hiding this comment

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

This implies all test in here are unit tests?

Copy link
Collaborator Author

@sidz sidz Jul 24, 2025

Choose a reason for hiding this comment

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

yup.

In my projects I usually use different test suites instead of groups. so Unit and Functional tests are placed in the different folders, ie:

tests/Unit
tests/Functional

@ruflin
Copy link
Owner

ruflin commented Jul 24, 2025

Nice update. Should we mention this in the changelog? Not sure TBH.

Copy link
Owner

@ruflin ruflin left a comment

Choose a reason for hiding this comment

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

Approving the change itself. Lets have a quick chat about the changelog entry.

@sidz
Copy link
Collaborator Author

sidz commented Jul 24, 2025

Nice update. Should we mention this in the changelog? Not sure TBH.

It makes sense @ruflin. I forgot to add a new line into Changelog.

Changelog updated

@ruflin ruflin merged commit 8ac84b7 into ruflin:9.x Jul 24, 2025
6 checks passed
@ruflin
Copy link
Owner

ruflin commented Jul 24, 2025

Thanks!

@sidz sidz deleted the update-phpstan branch July 24, 2025 09:07
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.

2 participants