Daco Harkes

> Note that as of #4068 we no longer analyze `test/` as part of publishing validation. I don't believe this is true. ``` $ pwd /Users/dacoharkes/src/dart-lang/native/pkgs/native_assets_builder $ dart pub publish...

It's a different validator `StrictDependenciesValidator` ```dart /// Validates that no Dart files in `benchmark/`, `test/` or /// `tool/` have dependencies that aren't in [deps] or [devDeps]. void _validateBenchmarkTestTool(Set deps, Set...

@DanTup What is the logic in DartCode?

I have lost context of what workflow I had when I was filing this issue. (And I'm mostly work from VS Code with Dart Code where I can rely on...

`dart pub publish --dry-run` implicitly runs a `dart pub get` in the main folder, and subsequently ignores `analysis_options.yaml` `exclude`s, causing https://github.com/dart-lang/ecosystem/issues/150. `dart pub publish --dry-run` listening to `analysis_options.yaml` would enable...

Cross-linking from the ideas in https://github.com/dart-lang/native/issues/39: We should consider a script instead of a yaml config as it should support conditional configuration. (E.g. only including something in debug mode, or...

Right, but then still you'd probably want to nest under the "target" package: ```json { "configVersion": 2, "packages": [ { "name": "bar", "rootUri": "$PUB_CACHE/...", "packageUri": "lib/", "languageVersion": "3.0" "config": {...

> Yeah I think that -D defines are typically invocation specific config, and also global, so I am not sure the pubspec is a good place for it. I was...

In a discussion with @mkustermann today we discussed we might actually want dynamic input in some cases (based on the build in build.dart) for tree shaking. So maybe we should...

That requires us to do an extra process invocation, this could slow down things. A process invocation is likely slower than just a programmatic source code invocation. In which case...