issues
search
meilisearch
/
meilisearch-rails
Meilisearch integration for Ruby on Rails
https://www.meilisearch.com
MIT License
295
stars
48
forks
source link
Refactor `meilisearch.rb`
#348
Open
ellnix
opened
6 months ago
ellnix
commented
6 months ago
[ ] Streamline interface
[ ] remove similarly named instance and class methods where one often just calls the other
[ ] reduce method side effects
[ ] reduce hacks in methods
[ ] clarify bang method API
[ ] do not use flag parameters (i. e.
product.index!(true)
)
[ ] make return types more consistent
[ ] Reach 95%+ code coverage
[ ] More robust error reporting (no failing silently)
[ ] Break
meilisearch.rb
into multiple files/modules
[ ] Add a system for validating configurations
product.index!(true)
)meilisearch.rb
into multiple files/modules