fix(lint): ignore clippy result_large_err for now#254
Conversation
There was a problem hiding this comment.
Pull Request Overview
This PR temporarily disables the clippy lint for large error types across several files to bypass new failures observed after the recent Rust upgrade.
- Temporarily allows clippy::result_large_err in multiple modules.
- Applies lint suppression in four different files.
Reviewed Changes
Copilot reviewed 4 out of 4 changed files in this pull request and generated 4 comments.
| File | Description |
|---|---|
| testing/test_actors/actors/frc46_factory_token/token_impl/src/lib.rs | Added lint allow attribute to ignore clippy::result_large_err. |
| testing/test_actors/actors/frc46_factory_token/src/lib.rs | Added lint allow attribute to ignore clippy::result_large_err. |
| testing/test_actors/actors/basic_token_actor/src/lib.rs | Added lint allow attribute to ignore clippy::result_large_err. |
| frc46_token/src/token/mod.rs | Added lint allow attribute to ignore clippy::result_large_err. |
| @@ -1,3 +1,5 @@ | |||
| #![allow(clippy::result_large_err)] | |||
There was a problem hiding this comment.
Consider adding an inline comment to explain why the clippy lint for large errors is being ignored, including a reference to any related issue for future tracking.
| @@ -1,3 +1,5 @@ | |||
| #![allow(clippy::result_large_err)] | |||
There was a problem hiding this comment.
Consider adding an inline comment to explain why the clippy lint for large errors is being ignored, including a reference to any related issue for future tracking.
| @@ -1,3 +1,5 @@ | |||
| #![allow(clippy::result_large_err)] | |||
There was a problem hiding this comment.
Consider adding an inline comment to explain why the clippy lint for large errors is being ignored, including a reference to any related issue for future tracking.
| @@ -1,3 +1,5 @@ | |||
| #![allow(clippy::result_large_err)] | |||
There was a problem hiding this comment.
Consider adding an inline comment to explain why the clippy lint for large errors is being ignored, including a reference to any related issue for future tracking.
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #254 +/- ##
=======================================
Coverage 89.00% 89.00%
=======================================
Files 27 27
Lines 5765 5765
=======================================
Hits 5131 5131
Misses 634 634
🚀 New features to boost your workflow:
|
3 participants
not sure why this is failing now and not when the last rust upgrade happened