Enable dangling_library_doc_comments and library_annotations lints (#117365)

Author: goderbauer

analysis_options.yaml

@@ -95,7 +95,7 @@ linter:
     # - constant_identifier_names # needs an opt-out https://github.com/dart-lang/linter/issues/204
     - control_flow_in_finally
     - curly_braces_in_flow_control_structures
-    # - dangling_library_doc_comments # not yet tested
+    - dangling_library_doc_comments
     - depend_on_referenced_packages
     - deprecated_consistency
     # - diagnostic_describe_all_properties # enabled only at the framework level (packages/flutter/lib)
@@ -116,7 +116,7 @@ linter:
     - iterable_contains_unrelated_type
     # - join_return_with_assignment # not required by flutter style
     - leading_newlines_in_multiline_strings
-    # - library_annotations # not yet tested
+    - library_annotations
     - library_names
     - library_prefixes
     - library_private_types_in_public_api

dev/bots/test/analyze-snippet-code-test-input/known_broken_documentation.dart

@@ -166,3 +166,4 @@
 /// error (something about backticks)
 /// this must be the last error, since it aborts parsing of this file
 /// ```
+String? foo;

dev/bots/test/analyze-snippet-code-test-input/short_but_still_broken.dart

@@ -11,7 +11,9 @@
 /// ```dart
 /// print(x);
 /// ```
+String? bar;
 
 /// error: empty dart block
 /// ```dart
 /// ```
+String? foo;

dev/bots/test/analyze-test-input/root/packages/foo/golden_doc.dart

@@ -35,7 +35,7 @@
 /// ```
 /// {@end-tool}
 ///
-
+String? foo;
 // Other comments
 // matchesGoldenFile('comment.png');
 

dev/bots/test/analyze-test-input/root/packages/foo/golden_ignore.dart

@@ -6,6 +6,7 @@
 // flutter_ignore_for_file: golden_tag (see analyze.dart)
 
 @Tags(<String>['some-other-tag'])
+library;
 
 import 'package:test/test.dart';
 

dev/bots/test/analyze-test-input/root/packages/foo/golden_missing_tag.dart

@@ -4,6 +4,7 @@
 
 // The reduced test set tag is missing. This should fail analysis.
 @Tags(<String>['some-other-tag'])
+library;
 
 import 'package:test/test.dart';
 

dev/bots/test/analyze_snippet_code_test.dart

@@ -29,7 +29,7 @@ const List<String> expectedMainErrors = <String>[
   'dev/bots/test/analyze-snippet-code-test-input/known_broken_documentation.dart:161:7: (top-level declaration) (undefined_identifier)',
   'dev/bots/test/analyze-snippet-code-test-input/known_broken_documentation.dart:165: Found "```" in code but it did not match RegExp: pattern=^ */// *```dart\$ flags= so something is wrong. Line was: "/// ```"',
   'dev/bots/test/analyze-snippet-code-test-input/short_but_still_broken.dart:9:12: (statement) (invalid_assignment)',
-  'dev/bots/test/analyze-snippet-code-test-input/short_but_still_broken.dart:17:4: Empty ```dart block in snippet code.',
+  'dev/bots/test/analyze-snippet-code-test-input/short_but_still_broken.dart:18:4: Empty ```dart block in snippet code.',
 ];
 
 const List<String> expectedUiErrors = <String>[

dev/bots/unpublish_package.dart

@@ -2,14 +2,14 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-
 /// This script removes published archives from the cloud storage and the
 /// corresponding JSON metadata file that the website uses to determine what
 /// releases are available.
 ///
 /// If asked to remove a release that is currently the release on that channel,
 /// it will replace that release with the next most recent release on that
 /// channel.
+library;
 
 import 'dart:async';
 import 'dart:convert';

dev/conductor/core/test/codesign_integration_test.dart

@@ -5,6 +5,7 @@
 // This test clones the framework and downloads pre-built binaries; it sometimes
 // times out with the default 5 minutes: https://github.com/flutter/flutter/issues/100937
 @Timeout(Duration(minutes: 10))
+library;
 
 import 'package:args/command_runner.dart';
 import 'package:conductor_core/src/codesign.dart' show CodesignCommand;

dev/devicelab/bin/tasks/flutter_gallery__back_button_memory.dart

@@ -2,9 +2,6 @@
 // Use of this source code is governed by a BSD-style license that can be
 // found in the LICENSE file.
 
-/// Measure application memory usage after pausing and resuming the app
-/// with the Android back button.
-
 import 'package:flutter_devicelab/framework/devices.dart';
 import 'package:flutter_devicelab/framework/framework.dart';
 import 'package:flutter_devicelab/framework/utils.dart';
@@ -13,6 +10,8 @@ import 'package:flutter_devicelab/tasks/perf_tests.dart';
 const String packageName = 'io.flutter.demo.gallery';
 const String activityName = 'io.flutter.demo.gallery.MainActivity';
 
+/// Measure application memory usage after pausing and resuming the app
+/// with the Android back button.
 class BackButtonMemoryTest extends MemoryTest {
   BackButtonMemoryTest() : super('${flutterDirectory.path}/dev/integration_tests/flutter_gallery', 'test_memory/back_button.dart', packageName);