Skip to content

Conversation

@CommanderStorm
Copy link
Contributor

@CommanderStorm CommanderStorm commented Jan 24, 2025

Pull Request

Related issue

no related issue, as just a docs change

What does this PR do?

  • Make doc-comments consistently be doc-comments
  • Also fixes a few links which cargo doc complained about

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!

@irevoire irevoire added the enhancement New feature or request label Jan 27, 2025
irevoire
irevoire previously approved these changes Jan 27, 2025
Copy link
Contributor

@irevoire irevoire left a comment

Choose a reason for hiding this comment

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

Thank you!

bors merge

meili-bors bot added a commit that referenced this pull request Jan 27, 2025
639: Make doc-comments consistently be doc-comments r=irevoire a=CommanderStorm

# Pull Request

## Related issue
no related issue, as just a docs change

## What does this PR do?
- Make doc-comments consistently be doc-comments 
- Also fixes a few links which `cargo doc` complained about

## 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: Frank Elsinga <frank@elsinga.de>
@meili-bors
Copy link
Contributor

meili-bors bot commented Jan 27, 2025

Build failed:

}

if expires_at.map_or(false, |expires_at| OffsetDateTime::now_utc() > expires_at) {
if expires_at.is_some_and(|expires_at| OffsetDateTime::now_utc() > expires_at) {
Copy link
Contributor Author

@CommanderStorm CommanderStorm Jan 27, 2025

Choose a reason for hiding this comment

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

@CommanderStorm
Copy link
Contributor Author

CommanderStorm commented Jan 27, 2025

sorry for the missed approach on this one. Did not think to run clippy (only cargo doc)

@curquiza curquiza added documentation Improvements or additions to documentation and removed enhancement New feature or request labels Feb 5, 2025
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

@meili-bors
Copy link
Contributor

meili-bors bot commented Feb 5, 2025

Build succeeded:

@meili-bors meili-bors bot merged commit 8575e24 into meilisearch:main Feb 5, 2025
7 checks passed
meili-bors bot added a commit that referenced this pull request Feb 18, 2025
645: Update version for the next release (v0.28) r=Strift a=meili-bot

_This PR is auto-generated._

The automated script updates the version of meilisearch-rust to a new version: "v0.28"

👇 CHANGELOG

## ✨ New

- [AI-powered search](https://www.meilisearch.com/docs/learn/ai_powered_search/getting_started_with_ai_search) is now stable `@/Strift`

## 🚀 Enhancements

* Remove in the next version non-existent experimental feature (#638) `@/CommanderStorm`
* Add support for language settings and search parameter (#620) `@/necocen`
* Support reverse param in get_tasks (#633) `@/ellnix`
* Add `indices` to `MatchRange` (#632) `@/LukasKalbertodt`
* Changes related to the next Meilisearch release (v1.13) (#637) 

## ⚙️ Maintenance/misc

* Use resolver = "2" (#623) `@/dureuill`
* Fix linting errors (#630) `@/Strift`
* Fix broken links (#636) `@/ellnix`
* Fix separator token code samples (#640) `@/ellnix`
* Make doc-comments consistently be doc-comments (#639) `@/CommanderStorm`

Thanks again to `@/CommanderStorm,` `@/LukasKalbertodt,` `@/Strift,` `@/dureuill,` `@/ellnix,` and `@n/ecocen!` 🎉


Co-authored-by: meili-bot <74670311+meili-bot@users.noreply.github.com>
Co-authored-by: Laurent Cazanove <lau.cazanove@gmail.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

documentation Improvements or additions to documentation

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants