Skip to content

Added support Laravel 12#36

Closed
rhoseno98 wants to merge 6 commits intomaize-tech:mainfrom
rhoseno98:main
Closed

Added support Laravel 12#36
rhoseno98 wants to merge 6 commits intomaize-tech:mainfrom
rhoseno98:main

Conversation

@rhoseno98
Copy link
Copy Markdown

No description provided.

- Added Maize Searchable support in the app.
- Configured searchable attributes for SourceCollection model.
- Updated composer autoload and providers to register the package.
- Added default match weight configuration.
- Changed timezone to Asia/Jakarta.

This improves search functionality by allowing customizable and weighted search attributes.
@enricodelazzari
Copy link
Copy Markdown
Member

Your changes look good, but could you please also update the GitHub Actions workflow to include tests for Laravel 12 and PHP 8.4?

You'll need to add 12.x to the laravel matrix and 8.4 to the php matrix in the .github/workflows/run-tests.yml file.

- Updated PHP version to 8.4 in phpstan and tests workflow.
- Added Laravel 12 and Testbench 10 to the test matrix.

This ensures compatibility with the latest PHP and Laravel versions.
@enricodelazzari
Copy link
Copy Markdown
Member

Just a couple more things:

  • Could you please revert the PHP version for the PHPStan workflow back to 8.1?
  • It looks like the tests are failing. I suspect this might be due to some dependency conflicts. Could you please take a look at the composer.json file and make the necessary adjustments to get the tests passing?

Let me know if you have any questions.

rhoseno added 4 commits June 23, 2025 18:05
- Refactored `AttributeUtilTest` to create `PostgresGrammar` with the DB connection for better clarity.
- Updated PHPStan configuration by removing `checkMissingIterableValueType`.
- Bumped development dependencies:
  - Updated Larastan to `^3.5.0`.
  - Updated Pest and related plugins to `^3.x`.
  - Updated Pint to `^1.22.1`.
  - Updated PHPStan-related packages to `^2.x`.
  - Extended Testbench compatibility to support version 10.

These changes enhance test readability, ensure compatibility with the latest tooling, and cleanup outdated configurations.
- Updated the `composer.json` file to require PHP ^8.2.
- Aligns package dependencies with the current PHP version support.
- Updated `composer.json` to support only Laravel 11 and 12.
- Removed support for older versions of Laravel and Testbench.
- Removed Laravel 10 and Testbench 8 from the matrix.
- Dropped PHP 8.1 support in tests.
- Simplified configurations by limiting compatibility to maintained versions.

Improves clarity and aligns testing with supported versions.
@enricodelazzari
Copy link
Copy Markdown
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