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 |
0 replies
|
cc @cgwalters looks like your code originally |
0 replies
|
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. |
0 replies
|
Ah fixture as in test fixture... OK, makes sense in that context |
0 replies
0 replies
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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
All reactions