search

meilisearch / meilisearch-java

Java client for Meilisearch
https://meilisearch.com
MIT License
187 stars 113 forks source link

Add CodeCov #641

Closed brunoocasali closed 1 year ago

brunoocasali commented 1 year ago

⚠️ This issue is generated, it means the examples and the namings do not necessarily correspond to the language of this repository. Also, if you are a maintainer, please add any clarification and instructions about this issue.

Sorry if this is already wholly/partially implemented. Feel free to let me know about the state of this issue in the repo.

Related to https://github.com/meilisearch/integration-guides/issues/59

Codecov helps us to know if a PR add non-tested code. If it's the case, we don't accept the PR. It's easy to install, easy to configure.

TODO:

StanislavMontag commented 1 year ago

Hey, Could you please assign this task to me?

brunoocasali commented 1 year ago

Hi @StanislavMontag

We prefer not assigning external people to our issues. Because people often ask to be assigned and never return, which discourages other potential contributions. We will accept and merge the first PR that fixes correctly and implements the issue.

tonyghouse commented 1 year ago

hi @brunoocasali . Raised PR for codecov config https://github.com/meilisearch/meilisearch-java/pull/644. check and suggest any changes if needed. have a doubt on the default test reports location in build.gradle. should i remove it or kept as it is?