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

Read parts with the defining compilation unit #728

Merged
merged 2 commits into from
Oct 10, 2024

Conversation

natebosch
Copy link
Member

@natebosch natebosch commented Oct 10, 2024

The direct read of parts from the library element is deprecated.
Retain ignores for reading imports and exports since they
will be restored without a deprecation.

Unblocks https://dart-review.googlesource.com/c/sdk/+/388643

The direct read of the imports, exports, and parts from the library
element is deprecated.

Unblocks https://dart-review.googlesource.com/c/sdk/+/388643
@jakemac53
Copy link
Contributor

It actually sounds like these are getting undeprecated... https://dart-review.googlesource.com/c/sdk/+/389142

@natebosch natebosch changed the title Read directives with the defining compilation unit Read parts with the defining compilation unit Oct 10, 2024
@natebosch natebosch merged commit 2bea904 into master Oct 10, 2024
12 checks passed
@natebosch natebosch deleted the analyzer-deprecations branch October 10, 2024 19:39
@@ -41,8 +41,7 @@ class LibraryReader {
...element.libraryImports,
Copy link
Contributor

Choose a reason for hiding this comment

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

I'm sorry, I failed to look precisely at it.
In analyzer 6.11 we un-deprecate LibraryElement.importedLibraries, not libraryImports.
So, these still should be migrated, like parts below.

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.

3 participants