Skip to content

[Class modifiers] Analyzer implementation #50728

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.

Sign up for GitHub

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

Jump to bottom
Closed
itsjustkevin opened this issue Dec 15, 2022 · 2 comments
Closed

[Class modifiers] Analyzer implementation #50728

itsjustkevin opened this issue Dec 15, 2022 · 2 comments
Assignees
@kallentu
Labels
legacy-area-analyzer Use area-devexp instead. P2 A bug or feature request we're likely to work on
Milestone
Dart 3 beta 2

Comments

@itsjustkevin
Copy link
Contributor

No description provided.

@itsjustkevin itsjustkevin mentioned this issue Dec 15, 2022
Open
@munificent munificent changed the title [NCM] Analyzer implementation [Class modifiers] Analyzer implementation Dec 17, 2022
@lrhn lrhn added the legacy-area-analyzer Use area-devexp instead. label Dec 20, 2022
@pq pq added the P2 A bug or feature request we're likely to work on label Dec 20, 2022
@itsjustkevin itsjustkevin added this to the Dart 3 beta 2 milestone Feb 6, 2023
@kallentu kallentu self-assigned this Feb 7, 2023
@kallentu
Copy link
Member

Quick update for the analyzer work:
All the work here is done except It is a compile-time error if a subtype of a declaration marked base or final is not marked base, final, or sealed which is the last loophole I need to close. Otherwise, everything functions as the spec expects.

@kallentu
Copy link
Member

kallentu commented Mar 2, 2023

The one last loophole landed as of today. Done all necessary work in analyzer.

@kallentu kallentu closed this as completed Mar 2, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@kallentu kallentu

Labels
legacy-area-analyzer Use area-devexp instead. P2 A bug or feature request we're likely to work on
Projects
None yet
Milestone
Dart 3 beta 2
Development

No branches or pull requests
4 participants
@pq @lrhn @itsjustkevin @kallentu