Skip to content

#2080. Move Correct member overrides to Superinterfaces folder #2096

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

Merged
merged 4 commits into from
Jun 22, 2023

Conversation

sgrekhov
Copy link
Contributor

11.2.2. Correct member overrides is part of 11.2. Superinterfaces chapter. Move tests to follow the specification structure

Copy link
Member

@eernstg eernstg left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Wow! A github diff that works! ;-)
LGTM

@eernstg
Copy link
Member

eernstg commented Jun 22, 2023

@sgrekhov, the PR is reported as having a conflict. I changed line 50 in Language/Interfaces/Superinterfaces/Correct_Member_Overrides/correct_member_overrides_A03_t06.dart, but that wasn't sufficient. Could you take a look? Presumably it's just another library which has been updated in master after the commit where this branch started.

@eernstg eernstg merged commit f4d2f72 into dart-lang:master Jun 22, 2023
copybara-service bot pushed a commit to dart-lang/sdk that referenced this pull request Jun 26, 2023
2023-06-23 sgrekhov22@gmail.com Fixes dart-lang/co19#2080. Fix typo in correct_member_overrides_A03_t14.dart (dart-lang/co19#2097)
2023-06-22 sgrekhov22@gmail.com dart-lang/co19#2080. Move `Correct member overrides` to Superinterfaces folder (dart-lang/co19#2096)
2023-06-22 sgrekhov22@gmail.com Fixes dart-lang/co19#2083. Expect `unspecified` error in const_collections_A08_t01.dart (dart-lang/co19#2084)
2023-06-22 sgrekhov22@gmail.com dart-lang/co19#2080. Update assertion texts, add mixin syntax test (dart-lang/co19#2095)
2023-06-22 sgrekhov22@gmail.com dart-lang/co19#2080. Add correct member overrides tests. Part 2. Setters (dart-lang/co19#2093)
2023-06-21 sgrekhov22@gmail.com Fixes dart-lang/co19#2090. Add required final instance variable to a inline class (dart-lang/co19#2091)
2023-06-20 sgrekhov22@gmail.com dart-lang/co19#2080. Add correct member overrides tests. Part 1. (dart-lang/co19#2088)
2023-06-20 sgrekhov22@gmail.com dart-lang/co19#2080. Update assertion texts of overriding tests (dart-lang/co19#2089)
2023-06-15 sgrekhov22@gmail.com dart-lang/co19#1400. Add inline classes members override test (dart-lang/co19#2085)

Change-Id: I4a5a112cb11590b0c15032a65f5b80193a42b27b
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/311160
Commit-Queue: Alexander Thomas <athom@google.com>
Reviewed-by: Alexander Thomas <athom@google.com>
@sgrekhov sgrekhov deleted the co19-2080-5 branch July 18, 2023 11:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants