Skip to content

Conversation

@stereotype441
Copy link
Member

The getters in the analyzer's AST data structures will soon be changed to less specific types, so we need to use the new interface in order to avoid warnings.

The getters in the analyzer's AST data structures will soon be changed to less specific types, so we need to use the new interface in order to avoid warnings.
// for details. All rights reserved. Use of this source code is governed by a
// BSD-style license that can be found in the LICENSE file.
import 'package:analyzer/dart/ast/ast.dart';
import 'package:analyzer/dart/ast/standard_resolution_map.dart';
Copy link
Member

Choose a reason for hiding this comment

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

Does this effectively bump the minimum pkg/analyzer version?

Any pubspec changes required?

Copy link
Member Author

Choose a reason for hiding this comment

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

Whoops, you're right. I forgot to update that. Fixing...

Copy link
Member

Choose a reason for hiding this comment

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

Cool!

Copy link
Member

@kevmoo kevmoo left a comment

Choose a reason for hiding this comment

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

:shipit:

Please do a squash merge when you land this.

@stereotype441 stereotype441 merged commit 8c78650 into master Jan 9, 2017
@stereotype441 stereotype441 deleted the use-resolutionMap branch January 9, 2017 20:13
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…_gen#121)

The getters in the analyzer's AST data structures will soon be changed to less specific types, so we need to use the new interface in order to avoid warnings.
mosuem pushed a commit to dart-lang/build that referenced this pull request Dec 10, 2024
…_gen#121)

The getters in the analyzer's AST data structures will soon be changed to less specific types, so we need to use the new interface in order to avoid warnings.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants