-
Notifications
You must be signed in to change notification settings - Fork 33
Add phpmd to project #165
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
Add phpmd to project #165
Conversation
28524b6
to
f411fe2
Compare
Seems that PHPMD tools have a problem and don't work across all versions, better deep understanding is needed on this. |
I moved this PR to draft @jonatanrdsantos so you can work without pressure 🕺 |
@jonatanrdsantos do you want to keep working in this PR? |
Hi @brunoocasali , no you can close this or pass to another one. |
Closing due inactivity :) Feel free to re-open @jonatanrdsantos 💪 |
f411fe2
to
233728c
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Since we removed the dependency on illuminate libraries, I just rebased your branch, and now your configuration works!
Thanks!
bors merge |
252: Update version for the next release (v0.12.0) r=brunoocasali a=meili-bot _This PR is auto-generated._ The automated script updates the version of meilisearch-symfony to a new version: "v0.12.0" **RELEASE CHANGELOG:** ## 💥 Breaking Changes * Accept a `normalizationGroups` as an array instead of the boolean `useSerializerGroup` when using `SearchableEntity` directly (#239) `@norkunas` * Allow configuring in `config/meilisearch.yml` a custom serializer groups instead of the default one (#239) `@norkunas` Example: ```yml enable_serializer_groups: true serializer_groups: ['public', 'private'] ``` ## 🚀 Enhancements * Start fixing PHPMD issues (#250) `@brunoocasali` ## 💅 Misc * Add phpmd to project (#165) `@jonatanrdsantos` Thanks again to `@brunoocasali,` `@curquiza,` `@jonatanrdsantos,` `@norkunas!` 🎉 Co-authored-by: meili-bot <[email protected]>
Pull Request
What does this PR do?
Fixes #164
PR checklist
Please check if your PR fulfills the following requirements:
Thank you so much for contributing to Meilisearch!