Remove linter errors (in `.rubocop_todo.yml `) about meilisearch-rails HOT 8 OPEN

Hello @AudTheCodeWitch

Thanks for your interest in this project :)

FYI, we prefer not assigning people to our issues because sometimes people ask to be assigned and never come back, which discourages the volunteer contributors to open a PR to fix this issue.
We will accept and merge the first PR that fixes correctly and well implements the issue following our contributing guidelines.

We are looking forward to reviewing your PR 🙂

from meilisearch-rails.

Hello @brunoocasali
thanks for these suggestions! 😄

We should create separated issues for what you suggest (the 3rd last points) since it's not related to this issue "Remove linter errors". Feel free to open them to detail what you plan to do a little bit 🙂
One issue per idea is ok.

from meilisearch-rails.

I would like to work on this issue, if possible.

from meilisearch-rails.

Hi @curquiza I'm interested to work on this. Hope I can get some advice on a failing spec while running on local for spec/configuration_spec.rb. Below is what I ran:

1. Ran ./meilisearch in one terminal
2. While checked out on the main branch of meilisearch-rails, I ran bundle exec rspec spec/configuration_spec.rb
3. I got the errors shown below:

Please let me know where I went wrong in running the steps. I unfortunately do not have docker installed on my laptop. Thank you so much!

from meilisearch-rails.

Hello @reginaalyssa
Did you run the required commands you can find in the CONTRIBUTING.md before running the spec?

from meilisearch-rails.

@curquiza I was sending many PR's but I didn’t mention what I want with them, the current approach is to:

• Fix small problems like double quote strings and others (Code/Style ## PR's)
• Fix most of the auto-correctable offenses (or all) (Code/Style ## PR's)
• Replace should with expect inside specs and offenses without changes on what the tests do.
• Start decomposing the big methods inside of the meilisearch-rails in order to fix the cyclomatic complexity
• Make tests independent (add randomization)

(feel free to reply with your concerns)

Probably after this, the rubocop_todo would be gone!

from meilisearch-rails.

Yes! I'll provide the issues ASAP :)

from meilisearch-rails.

Thanks a lot

from meilisearch-rails.