Chromium Code Reviews
chromiumcodereview-hr@appspot.gserviceaccount.com (chromiumcodereview-hr) | Please choose your nickname with Settings | Help | Chromium Project | Gerrit Changes | Sign out
(117)

Unified Diff: README.md

Issue 1535893002: Change the deferred library check tool to allow unspecified packages to be in (Closed) Base URL: git@github.com:dart-lang/dart2js_info.git@master
Patch Set: Created 5 years ago
Use n/p to move between diff chunks; N/P to move between comments. Draft comments are only viewable by you.
Jump to:
View side-by-side diff with in-line comments
Download patch
Index: README.md
diff --git a/README.md b/README.md
index 524a38b8286352a65dee6a1d2c0f752e5961e96c..ffc1ce885ec180b1a57042e5b9ba9d7ad9a0d305 100644
--- a/README.md
+++ b/README.md
@@ -207,32 +207,38 @@ The format of the YAML file is:
```yaml
main:
- packages:
+ include:
- some_package
Siggi Cherem (dart-lang) 2015/12/18 02:29:41 I like that the new syntax allows you to extend th
Harry Terkelsen 2015/12/18 21:22:21 Right, we can add that when customers want to chec
- other_package
+ exclude:
+ - some_other_package
foo:
- packages:
+ include:
- foo
- bar
baz:
- packages:
+ include:
- baz
- quux
+ exclude:
+ - zardoz
```
The YAML file consists of a list of declarations, one for each deferred
part expected in the output. At least one of these parts must be named
"main"; this is the main part that contains the program entrypoint. Each
top-level part contains a list of package names that are expected to be
-contained in that part. Any package that is not explicitly listed is
-expected to be in the main part. For instance, in the example YAML above
-the part named "baz" is expected to contain the packages "baz" and "quux".
+contained in that part, a list of package names that are expected to be in
+another part, or both. For instance, in the example YAML above the part named
+"baz" is expected to contain the packages "baz" and "quux" and exclude the
+package "zardoz".
The names for parts given in the specification YAML file (besides "main")
-are arbitrary and just used for reporting when the output does not meet the
-specification.
+are the same as the name given to the deferred import in the dart file. For
+instance, if you have `import 'package:foo/foo.dart' deferred as foo;` in your
Siggi Cherem (dart-lang) 2015/12/18 02:29:41 to make it less ambiguous, it might be worth chang
Harry Terkelsen 2015/12/18 21:22:21 It's actually the other way. The part after 'as' i
Siggi Cherem (dart-lang) 2015/12/18 21:34:25 I see, thanks for clarifying, that's nice, for som
+dart file, then the corresponding name in the specification file is 'foo'.
### Function size analysis tool
« no previous file with comments | « CHANGELOG.md ('k') | lib/deferred_library_check.dart » ('j') | lib/deferred_library_check.dart » ('J')

Powered by Google App Engine
This is Rietveld 408576698