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.

Sign up for GitHub

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

Jump to bottom

Standards #4

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Standards #4

wants to merge 4 commits into from

Conversation

lloricode
Copy link

@lloricode lloricode commented Sep 25, 2023 •
edited
Loading

  • set php 8.1 minimum support (since you used php enum feature)
  • use spatie/laravel-package-tools
  • add illuminate/database and illuminate/support dependencies with laravel "^9.0|^10.0" support
  • use laravel/pint for code style
  • add composer script for formatting code

@lloricode lloricode marked this pull request as draft September 25, 2023 01:17
@lloricode
Install laravel pint as formatter
18bc741 
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode
wip
68beaf0 
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode
wip
d6b452b 
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@lloricode lloricode marked this pull request as ready for review September 25, 2023 10:07
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
@lloricode