Replies: 5 comments
-
Clippy is complaining about this line. While trying to fix the clippy diagnostic, I tried to understand what the line even means, but it really seems non-sensical |
Beta Was this translation helpful? Give feedback.
-
cc @cgwalters looks like your code originally |
Beta Was this translation helpful? Give feedback.
-
Clearly that code needed a comment, I don't remember for sure why I did that. But just staring at it my vague guess is that it was a way to vary the labels in a deterministic way, i.e. not everything in So we can trivially change it to something that doesn't trigger a clippy lint. BTW note this is just in test code. |
Beta Was this translation helpful? Give feedback.
-
Ah fixture as in test fixture... OK, makes sense in that context |
Beta Was this translation helpful? Give feedback.
-
Beta Was this translation helpful? Give feedback.
Uh oh!
There was an error while loading. Please reload this page.
Uh oh!
There was an error while loading. Please reload this page.
-
... seems unimportant
Is this a bug / typo?
https://github.com/containers/bootc/blob/a3731457a22ffc0d412b3529a0a321a7031e7468/ostree-ext/src/fixture.rs#L247
Beta Was this translation helpful? Give feedback.
All reactions