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

Update readme links to the documentation #1650

Merged
merged 1 commit into from
May 13, 2024
Merged

Conversation

mdubus
Copy link
Member

@mdubus mdubus commented May 7, 2024

Pull Request

What does this PR do?

Update some links to the documentation (URL was wrong or not existing)

PR checklist

Please check if your PR fulfills the following requirements:

  • Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
  • 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!

Copy link

codecov bot commented May 7, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 97.54%. Comparing base (4475864) to head (ba8e617).

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1650   +/-   ##
=======================================
  Coverage   97.54%   97.54%           
=======================================
  Files          22       22           
  Lines         857      857           
  Branches       93       86    -7     
=======================================
  Hits          836      836           
  Misses         21       21           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@mdubus mdubus requested a review from curquiza May 7, 2024 13:29
@curquiza curquiza added the maintenance Issue about maintenance (CI, tests, refacto...) label May 7, 2024
Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

Niiiiice

thank you!

bors merge

meili-bors bot added a commit that referenced this pull request May 7, 2024
1650: docs: update readme links to the documentation r=curquiza a=mdubus

# Pull Request

## What does this PR do?
Update some links to the documentation (URL was wrong or not existing)

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Morgane Dubus <morgane.d@meilisearch.com>
Copy link
Contributor

meili-bors bot commented May 7, 2024

Build failed:

@curquiza
Copy link
Member

curquiza commented May 7, 2024

bors merge

meili-bors bot added a commit that referenced this pull request May 7, 2024
1650: docs: update readme links to the documentation r=curquiza a=mdubus

# Pull Request

## What does this PR do?
Update some links to the documentation (URL was wrong or not existing)

## PR checklist
Please check if your PR fulfills the following requirements:
- [x] Does this PR fix an existing issue, or have you listed the changes applied in the PR description (and why they are needed)?
- [x] Have you read the contributing guidelines?
- [x] Have you made sure that the title is accurate and descriptive of the changes?

Thank you so much for contributing to Meilisearch!


Co-authored-by: Morgane Dubus <morgane.d@meilisearch.com>
Copy link
Contributor

meili-bors bot commented May 7, 2024

Build failed:

Copy link
Member

@curquiza curquiza left a comment

Choose a reason for hiding this comment

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

bors merge

@curquiza curquiza changed the title docs: update readme links to the documentation Update readme links to the documentation May 13, 2024
Copy link
Contributor

meili-bors bot commented May 13, 2024

Build succeeded:

@meili-bors meili-bors bot merged commit 2f37b74 into main May 13, 2024
7 checks passed
@meili-bors meili-bors bot deleted the update_readme_links branch May 13, 2024 08:14
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
maintenance Issue about maintenance (CI, tests, refacto...)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants