Update Rust #1015

Merged
KitaitiMakoto merged 4 commits from bump-rust into main 2 years ago
Owner
<!-- We would appreciated if you report a bug at our Gitea instance's pull request page: https://git.joinplu.me/Plume/Plume/pulls You can login to the Gitea with your GitHub account. We welcome to receive pull requests here, GitHub, too. -->
KitaitiMakoto added 2 commits 2 years ago
KitaitiMakoto added 1 commit 2 years ago
KitaitiMakoto added 1 commit 2 years ago
KitaitiMakoto merged commit 57a54cf016 into main 2 years ago
KitaitiMakoto deleted branch bump-rust 2 years ago
igalic reviewed 2 years ago
igalic left a comment
Owner

@ -58,3 +58,3 @@
.map(|(offset, _)| offset)
.next()
.unwrap_or_else(|| self.text.len())
.unwrap_or(self.text.len())
igalic commented 2 years ago
Owner

how come this is changed to unwrap_or()

how come this is changed to `unwrap_or()`
@ -18,3 +18,3 @@
@media.alt_text
} else {
@media.content_warning.unwrap_or(i18n!(ctx.1, "No description"))
@media.content_warning.unwrap_or_else(|| i18n!(ctx.1, "No description")))
igalic commented 2 years ago
Owner

but this is changed to unwrap_or_else()?

but this is changed to `unwrap_or_else()`?
Poster
Owner
I wordered, too. But cargo clippy says to change so: * https://app.circleci.com/pipelines/github/Plume-org/Plume/1829/workflows/0ed62fc2-67e2-4baa-8806-2503d8831370/jobs/15297 * https://app.circleci.com/pipelines/github/Plume-org/Plume/1829/workflows/0ed62fc2-67e2-4baa-8806-2503d8831370/jobs/15298 I don't care these slight differences and avoided investigating why clippy works so more.
The pull request has been merged as 57a54cf016.
Sign in to join this conversation.
No reviewers
No Milestone
No Assignees
2 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: Plume/Plume#1015
Loading…
There is no content yet.