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

Unified Diff: pkg/polymer/lib/transformer.dart

Issue 179163007: [polymer] skip linter in tests (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: rebase Created 6 years, 10 months 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: pkg/polymer/lib/transformer.dart
diff --git a/pkg/polymer/lib/transformer.dart b/pkg/polymer/lib/transformer.dart
index 9e27906498dcf2ad688c6da2bf9c84efe7f8a899..717ae51764bc0f78265be19c1f48f97ed9445066 100644
--- a/pkg/polymer/lib/transformer.dart
+++ b/pkg/polymer/lib/transformer.dart
@@ -66,15 +66,16 @@ _readEntrypoints(value) {
}
/// Create deploy phases for Polymer. Note that inlining HTML Imports
-/// comes first (other than linter), which allows the rest of the
-/// HTML-processing phases to operate only on HTML that is actually imported.
+/// comes first (other than linter, if [options.linter] is enabled), which
+/// allows the rest of the HTML-processing phases to operate only on HTML that
+/// is actually imported.
List<List<Transformer>> _createDeployPhases(TransformOptions options) {
- return [
- [new Linter(options)],
+ var phases = options.lint ? [[new Linter(options)]] : [];
+ return phases..addAll([
[new ImportInliner(options)],
[new ObservableTransformer()],
[new ScriptCompactor(options)],
[new PolyfillInjector(options)],
[new BuildFilter(options)]
- ];
+ ]);
}
« pkg/polymer/lib/src/build/common.dart ('K') | « pkg/polymer/lib/src/build/common.dart ('k') | no next file » | no next file with comments »

Powered by Google App Engine
This is Rietveld 408576698