Skip to content
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

Meilisearch extra #316

Merged
merged 3 commits into from
Jun 19, 2021
Merged

Meilisearch extra #316

merged 3 commits into from
Jun 19, 2021

Conversation

molfar
Copy link
Contributor

@molfar molfar commented Jun 19, 2021

This is fixed PR for Meilisearch extras as a replacement for #313
@ddnexus I took into account you comments about code style and fixed them. But with docs I have some difficulties with Vscode, which makes some extra changes to markdown layout (some extra spaces, asterisk replacements etc). I have no experience with markdown at all, thats why I dont include docs commit here.

@molfar molfar mentioned this pull request Jun 19, 2021
@codecov
Copy link

codecov bot commented Jun 19, 2021

Codecov Report

Merging #316 (5188287) into dev (fb65260) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff            @@
##               dev      #316   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           45        47    +2     
  Lines         1794      1837   +43     
=========================================
+ Hits          1794      1837   +43     
Impacted Files Coverage Δ
lib/pagy/extras/meilisearch.rb 100.00% <100.00%> (ø)
test/mock_helpers/meilisearch.rb 100.00% <100.00%> (ø)

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update fb65260...5188287. Read the comment docs.

@ddnexus
Copy link
Owner

ddnexus commented Jun 19, 2021

I saw that extra md formatting with VSCode. You should be able to avoid it by disabling the format-on-save for markdown in the VSCode settings. This post should help.

Copy link
Owner

@ddnexus ddnexus left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just missing a few line sorting and alignments

README.md Outdated Show resolved Hide resolved
docs/_layouts/default.html Outdated Show resolved Hide resolved
docs/extras.md Outdated Show resolved Hide resolved
tasks/test.rake Outdated Show resolved Hide resolved
@ddnexus ddnexus merged commit 744717f into ddnexus:dev Jun 19, 2021
@ddnexus
Copy link
Owner

ddnexus commented Jun 19, 2021

Nice contribution! Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants