cedar-policy / cedar

Implementation of the Cedar Policy Language
https://www.cedarpolicy.com
Apache License 2.0
892 stars 80 forks source link

Don't warn on ascii idents #1336

Closed john-h-kastner-aws closed 4 hours ago

john-h-kastner-aws commented 3 days ago

This still warns on ascii control characters and any non-ascii characters we previosly warned about.

Fixes #621

Description of changes

Issue #, if available

Checklist for requesting a review

The change in this PR is (choose one, and delete the other options):

I confirm that this PR (choose one, and delete the other options):

I confirm that cedar-spec (choose one, and delete the other options):

I confirm that docs.cedarpolicy.com (choose one, and delete the other options):