Skip to content

Conversation

keertip
Copy link
Collaborator

@keertip keertip commented Feb 6, 2025

Migrate to Element2

  • remove 'element' from ModelElement/Warnable
  • lib/src/model/accessor.dart
  • lib/src/model/field.dart
  • lib/src/model/extension_type.dart
  • test/end2end/model_test.dart

Was looking to also rename element2 to element, but this was not possible as it there are some local variables also named element. Hence the renames in the PR, but this is still not the whole set. Will do the rename in a later PR.

@keertip keertip requested a review from srawlins February 6, 2025 19:42
Copy link
Member

@srawlins srawlins left a comment

Choose a reason for hiding this comment

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

Love it!

@keertip keertip merged commit 75d1fd7 into dart-lang:main Feb 6, 2025
9 checks passed
@keertip keertip deleted the migrateModel branch February 6, 2025 21:56
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