dart-sdk/pkg
Danny Tuppeny 8941837fb7 [analysis_server] Prevent "Extract Method" refactor showing up for function declarations
Fixes https://github.com/Dart-Code/Dart-Code/issues/4165.

Change-Id: I799220fffabfa39af1b2fd3039cb2de903f90c72
Reviewed-on: https://dart-review.googlesource.com/c/sdk/+/262348
Reviewed-by: Brian Wilkerson <brianwilkerson@google.com>
Commit-Queue: Brian Wilkerson <brianwilkerson@google.com>
2022-10-03 15:08:48 +00:00
..
_fe_analyzer_shared [parser] Field formal parameter with explicit record type 2022-10-03 07:14:26 +00:00
_js_interop_checks
analysis_server [analysis_server] Prevent "Extract Method" refactor showing up for function declarations 2022-10-03 15:08:48 +00:00
analysis_server_client
analyzer Adjust patterns parsing tests to follow analyzer conventions 2022-10-02 16:38:50 +00:00
analyzer_cli
analyzer_plugin
analyzer_utilities
async_helper
build_integration
compiler Type Socket in _HttpConnection to avoid dynamic invocations. 2022-09-30 19:29:56 +00:00
dart2js_info
dart2js_runtime_metrics
dart2js_tools
dart2native
dart2wasm
dart_internal
dartdev
dds
dds_service_extensions
dev_compiler
expect
front_end [cfe] Pass --no-sound-null-safety explicitly 2022-10-03 11:52:58 +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
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