Adds non_exhaustive to some error types that would be reasonable to extend in the future (IMO). What prompted this was that I wanted to backport #890 to 3.3.x, but found that I couldn't due to a new variant in SchemaError. This would have been fine if the error type had been non-exhaustive.
Also includes a quick fix to a comment in api.rs.
Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):
[X] A breaking change requiring a major version bump to cedar-policy (e.g., changes to the signature of an existing API).
I confirm that this PR (choose one, and delete the other options):
[X] Updates the "Unreleased" section of the CHANGELOG with a description of my change (required for major/minor version bumps).
I confirm that cedar-spec (choose one, and delete the other options):
[ ] Does not require updates because my change does not impact the Cedar formal model or DRT infrastructure.
[ ] Requires updates, and I have made / will make these updates myself. (Please include in your description a timeline or link to the relevant PR in cedar-spec, and how you have tested that your updates are correct.)
[ ] Requires updates, but I do not plan to make them in the near future. (Make sure that your changes are hidden behind a feature flag to mark them as experimental.)
[ ] I'm not sure how my change impacts cedar-spec. (Post your PR anyways, and we'll discuss in the comments.)
Description of changes
Adds
non_exhaustiveto some error types that would be reasonable to extend in the future (IMO). What prompted this was that I wanted to backport #890 to 3.3.x, but found that I couldn't due to a new variant inSchemaError. This would have been fine if the error type had been non-exhaustive.Also includes a quick fix to a comment in
api.rs.Issue #, if available
Checklist for requesting a review
The change in this PR is (choose one, and delete the other options):
cedar-policy(e.g., changes to the signature of an existing API).I confirm that this PR (choose one, and delete the other options):
I confirm that
cedar-spec(choose one, and delete the other options):cedar-spec, and how you have tested that your updates are correct.)cedar-spec. (Post your PR anyways, and we'll discuss in the comments.)