-
Notifications
You must be signed in to change notification settings - Fork 736
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
Fix BInvokableType
record fields of effective types not inheriting readonly
flag
#42582
base: master
Are you sure you want to change the base?
Fix BInvokableType
record fields of effective types not inheriting readonly
flag
#42582
Conversation
cdfb396
to
a4f21bd
Compare
BInvokableType
fields of effective types not inheriting readonly
flagBInvokableType
record fields of effective types not inheriting readonly
flag
This PR has been open for more than 15 days with no activity. This will be closed in 3 days unless the |
Closed PR due to inactivity for more than 18 days. |
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## master #42582 +/- ##
============================================
- Coverage 77.69% 77.68% -0.02%
+ Complexity 51483 51472 -11
============================================
Files 2933 2933
Lines 204235 204235
Branches 26716 26716
============================================
- Hits 158678 158651 -27
- Misses 36935 36958 +23
- Partials 8622 8626 +4 ☔ View full report in Codecov by Sentry. |
...rd/cloned_record_field_flag_inheritance/imported_module_with_readonly_records/Ballerina.toml
Outdated
Show resolved
Hide resolved
This PR has been open for more than 15 days with no activity. This will be closed in 3 days unless the |
Purpose
Fixes #42439
Approach
When generating the effective types, the original type is cloned. But during the cloning process the original flags of
BInvokableType
fields ofBRecordType
were not inherited properly. With this PR we fixed this inheritance issue.Samples
Remarks
Check List