[linter] Exclude ElementLocation from analyzer_use_new_elements + update migration file
As far as I can tell, `ElementLocation` isn't something we need to migrate off so this excludes it and updates the list with some additional files that no longer trigger the lint.
Change-Id: I31fae6616e0b0fa7e8a2f969ced74c5e8faacbff
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/390601
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
diff --git a/pkg/analysis_server/analyzer_use_new_elements.txt b/pkg/analysis_server/analyzer_use_new_elements.txt
index b65d862..7f77e69 100644
--- a/pkg/analysis_server/analyzer_use_new_elements.txt
+++ b/pkg/analysis_server/analyzer_use_new_elements.txt
@@ -132,6 +132,7 @@
lib/src/lsp/handlers/handler_change_workspace_folders.dart
lib/src/lsp/handlers/handler_code_actions.dart
lib/src/lsp/handlers/handler_code_lens.dart
+lib/src/lsp/handlers/handler_completion.dart
lib/src/lsp/handlers/handler_dart_text_document_content_provider.dart
lib/src/lsp/handlers/handler_document_color.dart
lib/src/lsp/handlers/handler_document_highlights.dart
@@ -154,6 +155,7 @@
lib/src/lsp/handlers/handler_signature_help.dart
lib/src/lsp/handlers/handler_states.dart
lib/src/lsp/handlers/handler_text_document_changes.dart
+lib/src/lsp/handlers/handler_type_hierarchy.dart
lib/src/lsp/handlers/handler_will_rename_files.dart
lib/src/lsp/handlers/handler_workspace_configuration.dart
lib/src/lsp/handlers/handler_workspace_symbols.dart
@@ -202,6 +204,7 @@
lib/src/server/stdio_server.dart
lib/src/services/completion/completion_performance.dart
lib/src/services/completion/completion_state.dart
+lib/src/services/completion/dart/dart_completion_suggestion.dart
lib/src/services/completion/dart/fuzzy_filter_sort.dart
lib/src/services/completion/dart/keyword_helper.dart
lib/src/services/completion/dart/label_helper.dart
@@ -296,7 +299,9 @@
lib/src/services/correction/dart/convert_to_null_aware.dart
lib/src/services/correction/dart/convert_to_null_aware_spread.dart
lib/src/services/correction/dart/convert_to_on_type.dart
+lib/src/services/correction/dart/convert_to_package_import.dart
lib/src/services/correction/dart/convert_to_raw_string.dart
+lib/src/services/correction/dart/convert_to_relative_import.dart
lib/src/services/correction/dart/convert_to_set_literal.dart
lib/src/services/correction/dart/convert_to_super_parameters.dart
lib/src/services/correction/dart/convert_to_switch_expression.dart
@@ -968,6 +973,7 @@
test/services/snippets/dart/test_all.dart
test/services/snippets/dart/test_definition_test.dart
test/services/snippets/dart/test_group_definition_test.dart
+test/services/snippets/dart/test_support.dart
test/services/snippets/dart/try_catch_statement_test.dart
test/services/snippets/dart/while_statement_test.dart
test/services/snippets/snippet_manager_test.dart
@@ -999,6 +1005,7 @@
test/src/computer/outline_computer_test.dart
test/src/computer/selection_range_computer_test.dart
test/src/computer/test_all.dart
+test/src/computer/type_hierarchy_computer_test.dart
test/src/domains/execution/completion_test.dart
test/src/domains/execution/test_all.dart
test/src/domains/flutter/base.dart
@@ -1420,6 +1427,7 @@
test/src/services/correction/fix/test_all.dart
test/src/services/correction/fix/update_sdk_constraints_test.dart
test/src/services/correction/fix/use_curly_braces_test.dart
+test/src/services/correction/fix/use_different_division_operator_test.dart
test/src/services/correction/fix/use_effective_integer_division_test.dart
test/src/services/correction/fix/use_eq_eq_null_test.dart
test/src/services/correction/fix/use_is_not_empty_test.dart
diff --git a/pkg/analysis_server/lib/src/services/completion/dart/dart_completion_suggestion.dart b/pkg/analysis_server/lib/src/services/completion/dart/dart_completion_suggestion.dart
index 6cf0998..1898818 100644
--- a/pkg/analysis_server/lib/src/services/completion/dart/dart_completion_suggestion.dart
+++ b/pkg/analysis_server/lib/src/services/completion/dart/dart_completion_suggestion.dart
@@ -4,7 +4,7 @@
import 'package:analysis_server/src/protocol_server.dart'
show CompletionSuggestion;
-import 'package:analyzer/dart/element/element.dart';
+import 'package:analyzer/dart/element/element2.dart';
/// An extension of [CompletionSuggestion] that includes additional
/// Dart-specific fields that are not part of the JSON protocol.
diff --git a/pkg/analyzer/lib/dart/element/element2.dart b/pkg/analyzer/lib/dart/element/element2.dart
index aeda1fb..309f4d7 100644
--- a/pkg/analyzer/lib/dart/element/element2.dart
+++ b/pkg/analyzer/lib/dart/element/element2.dart
@@ -75,7 +75,8 @@
DirectiveUriWithSource,
DirectiveUriWithUnit,
ElementAnnotation,
- ElementKind;
+ ElementKind,
+ ElementLocation;
/// An element or fragment that can have either annotations (metadata), a
/// documentation comment, or both associated with it.
diff --git a/pkg/linter/analyzer_use_new_elements.txt b/pkg/linter/analyzer_use_new_elements.txt
index 7963293..8376624 100644
--- a/pkg/linter/analyzer_use_new_elements.txt
+++ b/pkg/linter/analyzer_use_new_elements.txt
@@ -19,6 +19,7 @@
lib/src/rules/avoid_double_and_int_checks.dart
lib/src/rules/avoid_dynamic_calls.dart
lib/src/rules/avoid_empty_else.dart
+lib/src/rules/avoid_equals_and_hash_code_on_mutable_classes.dart
lib/src/rules/avoid_escaping_inner_quotes.dart
lib/src/rules/avoid_final_parameters.dart
lib/src/rules/avoid_function_literals_in_foreach_calls.dart
@@ -53,6 +54,7 @@
lib/src/rules/control_flow_in_finally.dart
lib/src/rules/curly_braces_in_flow_control_structures.dart
lib/src/rules/dangling_library_doc_comments.dart
+lib/src/rules/diagnostic_describe_all_properties.dart
lib/src/rules/directives_ordering.dart
lib/src/rules/discarded_futures.dart
lib/src/rules/do_not_use_environment.dart
@@ -173,6 +175,7 @@
lib/src/rules/unrelated_type_equality_checks.dart
lib/src/rules/use_colored_box.dart
lib/src/rules/use_decorated_box.dart
+lib/src/rules/use_full_hex_values_for_flutter_colors.dart
lib/src/rules/use_function_type_syntax_for_parameters.dart
lib/src/rules/use_if_null_to_convert_nulls_to_bools.dart
lib/src/rules/use_is_even_rather_than_modulo.dart
diff --git a/pkg/linter/lib/src/rules/analyzer_use_new_elements.dart b/pkg/linter/lib/src/rules/analyzer_use_new_elements.dart
index 55a2829..9a5b00e 100644
--- a/pkg/linter/lib/src/rules/analyzer_use_new_elements.dart
+++ b/pkg/linter/lib/src/rules/analyzer_use_new_elements.dart
@@ -29,6 +29,7 @@
'DirectiveUriWithSource',
'ElementAnnotation',
'ElementKind',
+ 'ElementLocation',
}.contains(firstFragment.name)) {
return false;
}
