Skip to content

Conversation

@crynobone
Copy link
Contributor

@crynobone crynobone commented Dec 26, 2024

  • use laravel/nova-devtool to improve local development
  • use npm instead of yarn (default with Laravel Nova 4 and 5)
  • format js syntax with npm run format
  • use Nova Workbench to handle tests and composer run serve to preview the package.

crynobone and others added 4 commits December 24, 2024 00:15
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
Signed-off-by: Mior Muhammad Zaki <[email protected]>
@crynobone crynobone marked this pull request as ready for review December 26, 2024 01:36
phpunit.xml.dist Outdated
<env name="DB_PORT" value="3306"/>
</php>
<php>
</php>
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Use sqlite by default. Run locally using composer test

- php: 8.4
laravel: 10.*
- php: 8.0
laravel: 11.*
Copy link
Contributor Author

Choose a reason for hiding this comment

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

Fix to run test on correct versions.

Signed-off-by: Mior Muhammad Zaki <[email protected]>
@freekmurze freekmurze merged commit 5b8c6e0 into spatie:main Dec 30, 2024
0 of 8 checks passed
@freekmurze
Copy link
Member

Thanks!

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