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

3 minor patches #274

Merged
merged 4 commits into from
Apr 1, 2022
Merged

3 minor patches #274

merged 4 commits into from
Apr 1, 2022

Conversation

cgwalters
Copy link
Member

cli: Add more docstrings

Specifically for commit.


cli: Use bool::then

Minor drive by code improvement.


tree-wide: Use some captured identifiers in format strings

xref https://blog.rust-lang.org/2022/01/13/Rust-1.58.0.html#captured-identifiers-in-format-strings
Another drive by code improvement, no immediate motivation. I've
just been using it in other places and I think we're fine to hard depend
on Rust 1.58 - it's in RHEL8.6 for example.


@@ -342,7 +342,7 @@ impl Chunking {
);
Cow::Owned(r)
}
n => Cow::Owned(format!("{} components", n)),
n => Cow::Owned(format!("{n} components")),
Copy link
Member

Choose a reason for hiding this comment

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

Oh sweet, will have to remember to start using this more now!

@jlebon
Copy link
Member

jlebon commented Mar 30, 2022

Need to bump MSRV here.

To use the new inline format bits.
Specifically for `commit`.
Minor drive by code improvement.
xref https://blog.rust-lang.org/2022/01/13/Rust-1.58.0.html#captured-identifiers-in-format-strings
Another drive by code improvement, no immediate motivation.  I've
just been using it in other places and I think we're fine to hard depend
on Rust 1.58 - it's in RHEL8.6 for example.
@cgwalters
Copy link
Member Author

Updated 🆕 MSRV

@cgwalters cgwalters merged commit 21f8920 into ostreedev:main Apr 1, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

2 participants