You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
@cdisselkoen noted that we could probably replace panic_safety.sh with the new clippy tools available in Rust 1.81 including the ability to do #[allow(clippy::expect_used, reason = "...")] and an option we can enforce on our workspace’s Cargo.toml to allow_attributes_without_reason = deny.
If this official tooling allows us to completely replace the panic safety script, we should do it.
Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?
👋 I may be able to implement this internal improvement
⚠️ This feature might incur a breaking change
The text was updated successfully, but these errors were encountered:
adpaco-aws
added
internal-improvement
Refactoring, minor performance improvement, or other changes that Cedar users may never notice
pending-triage
The cedar maintainers haven't looked at this yet. Automicaly added to all new issues.
labels
Feb 28, 2025
Describe the improvement you'd like to request
@cdisselkoen noted that we could probably replace
panic_safety.sh
with the new clippy tools available in Rust 1.81 including the ability to do#[allow(clippy::expect_used, reason = "...")]
and an option we can enforce on our workspace’s Cargo.toml toallow_attributes_without_reason = deny
.If this official tooling allows us to completely replace the panic safety script, we should do it.
Describe alternatives you've considered
No response
Additional context
No response
Is this something that you'd be interested in working on?
The text was updated successfully, but these errors were encountered: