Skip to content

Generate mixin after composer install/update#77

Merged
martinsoenen merged 1 commit intomainfrom
generate-mixin-on-install
Mar 14, 2026
Merged

Generate mixin after composer install/update#77
martinsoenen merged 1 commit intomainfrom
generate-mixin-on-install

Conversation

@martinsoenen
Copy link
Contributor

@martinsoenen martinsoenen commented Feb 5, 2026

This will generate the mixin during the installation of the package and get the autocompletion in the IDE.

I'm currently looking if xefi/faker-php-laravel & xefi/faker-php-symfony are compatible with what I've done before submiting the PR.

@GautierDele
Copy link
Member

Quite like the idea, is someting missing here ?

@martinsoenen
Copy link
Contributor Author

@GautierDele I need to try with the framework integrations to see if it's working well too

@martinsoenen
Copy link
Contributor Author

@GautierDele I need to try with the framework integrations to see if it's working well too

It seems post-install-cmd and post-update-cmd are executed only on a direct install of the package. So we'll need to create these scripts directly in framework integrations, and there will be no conflicts. Perfect for us. So can anyone review please ?

If it's ok for you we release :)

@martinsoenen martinsoenen marked this pull request as draft March 10, 2026 13:19
@martinsoenen martinsoenen marked this pull request as ready for review March 10, 2026 13:19
@martinsoenen martinsoenen changed the title Draft: Generate mixin after composer install/update Generate mixin after composer install/update Mar 10, 2026
Copy link
Member

@GautierDele GautierDele left a comment

Choose a reason for hiding this comment

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

LGTM ;)

@martinsoenen martinsoenen merged commit f8e0042 into main Mar 14, 2026
4 checks passed
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