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

Clean tests files for latest dependencies #59

Merged
merged 17 commits into from
Aug 14, 2023

Conversation

lloricode
Copy link
Contributor

@lloricode lloricode commented Aug 2, 2023

  • Add github action for test ci
  • Drop support PHP below v8.0
  • Drop support Laravel below v8.0
  • Convert PHPunit to PestPHP
  • Install laravel/pint (only run on tests folder for now)
  • Make database test real-time
  • Install spatie/laravel-package-tools for package standard

Removing EOL Laravel and PHP version for easy to maintain and use latest technology
see laravelversions.com and supported php versions

plan later if this merge

@lloricode lloricode marked this pull request as draft August 2, 2023 14:10
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 August 2, 2023 15:41
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
Signed-off-by: Lloric Mayuga Garcia <[email protected]>
@saqueib
Copy link
Member

saqueib commented Aug 14, 2023

Thank you so much @lloricode, merging it now.

@saqueib saqueib merged commit 04d3d4c into qcod:master Aug 14, 2023
@lloricode lloricode deleted the feature/clean-up branch August 14, 2023 04:40
@saqueib
Copy link
Member

saqueib commented Aug 15, 2023

@lloricode please check if you can do something about this one #58 . I can only see issue if we put same Name Badge in sub folder that will cause issue. What you think, anyway to make it work

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