dart-sdk/pkg
Sam Rawlins 15be61ed9a Remove INVALID_OVERRIDE_DIFFERENT_DEFAULT_VALUES warnings
Fixes https://github.com/dart-lang/sdk/issues/49112

It was decided to remove the "override with equal default value"
restriction both for null safe code, and for pre-null safe code.

CFE had never issued this static warning, and her we remove it from
analyzer.

Change-Id: I1244e4fe46da8bb4bd8c3a77ec8beb95811e30a1
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/262267
Reviewed-by: Konstantin Shcheglov <scheglov@google.com>
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
2022-10-04 18:23:10 +00:00
..
_fe_analyzer_shared
_js_interop_checks
analysis_server Remove INVALID_OVERRIDE_DIFFERENT_DEFAULT_VALUES warnings 2022-10-04 18:23:10 +00:00
analysis_server_client
analyzer Remove INVALID_OVERRIDE_DIFFERENT_DEFAULT_VALUES warnings 2022-10-04 18:23:10 +00:00
analyzer_cli
analyzer_plugin Deprecate CatchClause.exceptionParameter2/stackTraceParameter2 2022-10-04 15:58:11 +00:00
analyzer_utilities
async_helper
build_integration
compiler [dart2js] migrate io/kernel_source_information.dart 2022-10-04 17:08:00 +00:00
dart2js_info
dart2js_runtime_metrics
dart2js_tools
dart2native
dart2wasm
dart_internal
dartdev [ CLI ] Add support for --[no-]sound-null-safety to dart compile {jit-snapshot,kernel} 2022-10-04 17:03:29 +00:00
dds
dds_service_extensions
dev_compiler
expect
front_end [cfe,vm,wasm] Handle private injected members like augmentations 2022-10-04 14:43:43 +00:00
frontend_server
js
js_ast
js_runtime
js_shared
kernel
language_versioning_2_7_test
meta
modular_test
native_stack_traces
nnbd_migration Deprecate CatchClause.exceptionParameter2/stackTraceParameter2 2022-10-04 15:58:11 +00:00
scrape
smith
sourcemap_testing
status_file
telemetry
test_runner
testing
vm
vm_service
vm_snapshot_analysis
wasm_builder
.gitignore
BUILD.gn
OWNERS
pkg.status
README.md

Package validation

The packages in pkg/ are automatically validated on the LUCI CI bots. The validation is largely done by the tools/package_deps package; it can be tested locally via:

dart tools/package_deps/bin/package_deps.dart

Packages which are published

There are several packages developed in pkg/ which are published to pub. Validation of these packages is particularly important because the pub tools are not used for these packages during development; we get our dependency versions from the DEPS file. Its very easy for the dependencies specified in a package's pubspec file to get out of date wrt the packages and versions actually used.

In order to better ensure we're publishing correct packages, we validate some properties of the pubspec files on our CI system. These validations include:

  • that the dependencies listed in the pubspec are used in the package
  • that all the packages used by the source are listed in the pubspec
  • that we don't use relative path deps to pkg/ or third_party/ packages

Packages which are not published

For packages in pkg/ which we do not intend to be published, we put the following comment in the pubspec.yaml file:

# This package is not intended for consumption on pub.dev. DO NOT publish.
publish_to: none

These pubspecs are still validated by the package validation tool. The contents are more informational as the pubspecs for these packages are not consumed by the pub tool or ecosystem.

We validate:

  • that the dependencies listed in the pubspec are used in the package
  • that all the packages used by the source are listed in the pubspec
  • that a reference to a pkg/ package is done via a relative path dependency