Skip to content
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 mixins with superclass constraints, to the 'implementers' list. #3844

Merged
merged 1 commit into from
Aug 24, 2024

Conversation

srawlins
Copy link
Member

"Implementers" is loose. The list of "implementors" of a class (etc.), C, includes classes (etc.) that extend, implement, mix in, or use-as-a-superclass-constraint, C.

Fixes #3406

Also I move a lot of tests to the unit-test-style class_test.dart.


  • I’ve reviewed the contributor guide and applied the relevant portions to this PR.
Contribution guidelines:

Note that many Dart repos have a weekly cadence for reviewing PRs - please allow for some latency before initial review feedback.

"Implementers" is loose. The list of "implementors" of a class (etc.), _C_,
includes classes (etc.) that extend, implement, mix in, or
use-as-a-superclass-constraint, _C_.

Fixes dart-lang#3406
@srawlins
Copy link
Member Author

CC @dart-lang/analyzer-team

@srawlins srawlins merged commit f7e8def into dart-lang:main Aug 24, 2024
9 checks passed
@srawlins srawlins deleted the mixins-are-implementors branch August 24, 2024 16:46
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.

Mixins with superclass constraints should be listed on those superclasses' pages
2 participants