Skip to content

Standard #29

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

Open
wants to merge 16 commits into
base: master
Choose a base branch
from
Open

Conversation

lloricode
Copy link
Contributor

@lloricode lloricode commented Nov 3, 2023

  • Use laravel/pint for code style fixes
  • Drop support PHP 8.2 below (see php 8.1 will soon end of life https://php.watch/versions)
  • Use Pest for testing
  • Use phpstan for static analysis (max rule level 10)
  • Add github actions
  • Replace travis with github action workflow

Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode lloricode marked this pull request as draft November 3, 2023 07:33
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode lloricode changed the title Type System Standard Apr 26, 2025
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode lloricode marked this pull request as ready for review April 26, 2025 15:10
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode
Copy link
Contributor Author

hi @kevinkhill! hope you review this, thank you

@kevinkhill
Copy link
Owner

kevinkhill commented Apr 26, 2025 via email

@lloricode
Copy link
Contributor Author

been tested on production, and it is great

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