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

Github Actions: Add Phpstan #105

Merged
merged 14 commits into from
Nov 27, 2023
Merged

Github Actions: Add Phpstan #105

merged 14 commits into from
Nov 27, 2023

Conversation

sukhwinder33445
Copy link
Contributor

@sukhwinder33445 sukhwinder33445 commented Jun 19, 2023

  • Fix relevant level 2 issues and then increase the level to max (add baseline).

@sukhwinder33445 sukhwinder33445 self-assigned this Jun 19, 2023
@cla-bot cla-bot bot added the cla/signed CLA is signed by all contributors of a PR label Jun 19, 2023
@sukhwinder33445 sukhwinder33445 force-pushed the phpstan branch 2 times, most recently from d62c963 to 5594d1c Compare July 12, 2023 15:23
@sukhwinder33445 sukhwinder33445 force-pushed the phpstan branch 4 times, most recently from a9f6ce2 to 33a49b1 Compare October 23, 2023 11:48
@nilmerg nilmerg force-pushed the phpstan branch 7 times, most recently from d0a91e1 to 5375899 Compare November 16, 2023 12:36
phpstan is not run with an action anymore, as the action runs
it its own docker container and hence has no access to files
outside the repository root. A side-effect of this is, that
phpstan now **really** runs with the php version set up by
the matrix.
@nilmerg nilmerg merged commit 18f483c into main Nov 27, 2023
12 checks passed
@nilmerg nilmerg deleted the phpstan branch November 27, 2023 10:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
cla/signed CLA is signed by all contributors of a PR
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants