Skip to content

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

Merged
merged 2 commits into from
May 12, 2023

Conversation

jonatanrdsantos
Copy link

Pull Request

What does this PR do?

Fixes #164

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue?
  • Have you read the contributing guidelines?
  • Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!

@brunoocasali brunoocasali self-requested a review February 9, 2022 21:01
@jonatanrdsantos jonatanrdsantos force-pushed the feature/add-phpmd branch 5 times, most recently from 28524b6 to f411fe2 Compare February 9, 2022 21:51
@jonatanrdsantos
Copy link
Author

Seems that PHPMD tools have a problem and don't work across all versions, better deep understanding is needed on this.

@brunoocasali brunoocasali marked this pull request as draft February 16, 2022 21:36
@brunoocasali
Copy link
Member

I moved this PR to draft @jonatanrdsantos so you can work without pressure 🕺

@brunoocasali
Copy link
Member

@jonatanrdsantos do you want to keep working in this PR?

@brunoocasali brunoocasali added enhancement New feature or request stale Pull request or issue that has recieve no activity for a long time. labels Apr 13, 2022
@jonatanrdsantos
Copy link
Author

Hi @brunoocasali , no you can close this or pass to another one.

@brunoocasali
Copy link
Member

Closing due inactivity :)

Feel free to re-open @jonatanrdsantos 💪

@brunoocasali brunoocasali marked this pull request as ready for review May 12, 2023 19:40
Copy link
Member

@brunoocasali brunoocasali left a 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!

@brunoocasali
Copy link
Member

bors merge

@brunoocasali brunoocasali merged commit 386845e into meilisearch:main May 12, 2023
meili-bors bot added a commit that referenced this pull request May 17, 2023
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]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request stale Pull request or issue that has recieve no activity for a long time.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Add PHPMD to improve code quality
3 participants