-
Notifications
You must be signed in to change notification settings - Fork 137
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Add required mapping
keyword to entitlement mapping usages
#2883
Add required mapping
keyword to entitlement mapping usages
#2883
Conversation
mapping
keyword to entitlement mapping usages
Cadence Benchstat comparisonThis branch with compared with the base branch onflow:feature/stable-cadence commit 0bfd2c0 Collapsed results for better readability
|
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## feature/stable-cadence #2883 +/- ##
==========================================================
+ Coverage 79.97% 79.99% +0.02%
==========================================================
Files 344 344
Lines 81252 81314 +62
==========================================================
+ Hits 64979 65045 +66
+ Misses 13965 13961 -4
Partials 2308 2308
Flags with carried forward coverage won't be shown. Click here to find out more.
☔ View full report in Codecov by Sentry. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice!
I assumed the changes would be mostly syntactical, but there are also some changes to the checking unrelated (?) to the new mapping
keyword?
Co-authored-by: Bastian Müller <[email protected]>
Yeah, the changes should be entirely syntactic, with the exception of a few checker-related changes in converting ast-level access to semantic access, since we no longer need to disambiguate between a singleton entitlement and a map. |
…to sainati/entitlement-mapping-syntax
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Nice! Just a couple minor suggestions and questions
Implementation of onflow/flips#210
master
branchFiles changed
in the Github PR explorer