Skip to content

Comments

clippy disable std::result::Result, eliminate VortexExpect#6575

Draft
a10y wants to merge 2 commits intodevelopfrom
aduffy/std-result-no
Draft

clippy disable std::result::Result, eliminate VortexExpect#6575
a10y wants to merge 2 commits intodevelopfrom
aduffy/std-result-no

Conversation

@a10y
Copy link
Contributor

@a10y a10y commented Feb 18, 2026

Summary

Closes: #000

Testing

a10y added 2 commits February 18, 2026 09:20
Signed-off-by: Andrew Duffy <andrew@a10y.dev>
Signed-off-by: Andrew Duffy <andrew@a10y.dev>
@a10y
Copy link
Contributor Author

a10y commented Feb 18, 2026

This almost works, the biggest blocker I hit is I can't find a way to combine allow(disallowed_types) with derive(serde::Serialize), since the error is in the expansion. I think the only way to do it is to make a module-level allow which is kind of shitty

@a10y a10y mentioned this pull request Feb 18, 2026
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.

1 participant