-
Notifications
You must be signed in to change notification settings - Fork 103
Closed
Labels
papercutSmall annoyances in the Cedar SDK. Lower priority fixes than bugs. Smaller than a feature requestSmall annoyances in the Cedar SDK. Lower priority fixes than bugs. Smaller than a feature request
Description
Describe the improvement you'd like to request
According to this comment we return the HierarchyNotRespected
error in some cases where we could soundly type an expression as False
. This error was introduced (IIRC) to avoid having to update the Dafny type soundness proof.
Now that we've switched to using Lean, and tweaked the validation property being tested, I wonder if we could remove this error variant.
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
Metadata
Metadata
Assignees
Labels
papercutSmall annoyances in the Cedar SDK. Lower priority fixes than bugs. Smaller than a feature requestSmall annoyances in the Cedar SDK. Lower priority fixes than bugs. Smaller than a feature request