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
Take argument labels into account when declaring members for nested declarations #1717
Take argument labels into account when declaring members for nested declarations #1717
Conversation
Codecov Report
@@ Coverage Diff @@
## master #1717 +/- ##
==========================================
- Coverage 77.00% 76.99% -0.02%
==========================================
Files 292 292
Lines 61187 61212 +25
==========================================
+ Hits 47117 47130 +13
- Misses 12459 12469 +10
- Partials 1611 1613 +2
Flags with carried forward coverage won't be shown. Click here to find out more.
Continue to review full report at Codecov.
|
Cadence Benchstat comparisonThis branch with compared with the base branch onflow:master commit 56d6da0 Results
|
2653: Fix Cadence code r=turbolent a=turbolent In github.com/onflow/cadence#1717, the check of argument labels for nested constructors was fixed. Fix the previously accepted code, which is now rejected: - Update to the fixed core contracts - Add the missing argument label Co-authored-by: Bastian Müller <bastian@axiomzen.co>
2653: Fix Cadence code r=turbolent a=turbolent In github.com/onflow/cadence#1717, the check of argument labels for nested constructors was fixed. Fix the previously accepted code, which is now rejected: - Update to the fixed core contracts - Add the missing argument label Co-authored-by: Bastian Müller <bastian@axiomzen.co>
2653: Fix Cadence code r=turbolent a=turbolent In github.com/onflow/cadence#1717, the check of argument labels for nested constructors was fixed. Fix the previously accepted code, which is now rejected: - Update to the fixed core contracts - Add the missing argument label Co-authored-by: Bastian Müller <bastian@axiomzen.co> Co-authored-by: Daniel Sainati <sainatidaniel@gmail.com>
Description
When declaring the members for nested declarations, e.g. a composite declaration inside a contract, also take the argument labels of the nested declaration into account.
This was reported by @justjoolz, who also provided a great reproduction in https://github.com/justjoolz/struct-test 🙏
master
branchFiles changed
in the Github PR explorer