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

Unified Diff: utils/pub/command_list.dart

Issue 10214006: Refactor command code and add support for --help and --version. (Closed) Base URL: https://dart.googlecode.com/svn/branches/bleeding_edge/dart
Patch Set: Created 8 years, 8 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: utils/pub/command_list.dart
diff --git a/utils/pub/command_list.dart b/utils/pub/command_list.dart
index 973a4dedc48a7e41a72d928afbab7f4b578eed99..5079c9632223aaafe441e130fc3c3206c5c10950 100644
--- a/utils/pub/command_list.dart
+++ b/utils/pub/command_list.dart
@@ -3,13 +3,14 @@
// BSD-style license that can be found in the LICENSE file.
/** Handles the `list` pub command. */
-void commandList(PubOptions options, List<String> args) {
- // TODO(rnystrom): Validate args. Right now, this just lists the packages in
- // your cache.
- final cache = new PackageCache(options.cacheDir);
- cache.listAll().then((packages) {
- for (final package in packages) {
- print(package.name);
- }
- });
-}
+class ListCommand extends PubCommand {
+ String get description() => 'print the contents of repositories';
nweiz 2012/04/25 20:49:41 This isn't really accurate... currently what this
Bob Nystrom 2012/04/25 22:56:40 Right.
nweiz 2012/04/25 23:09:22 Oh, sure. Maybe just add a comment about that here
Bob Nystrom 2012/04/26 00:22:13 Done.
+
+ void onRun() {
+ cache.listAll().then((packages) {
+ for (final package in packages) {
+ print(package.name);
+ }
+ });
+ }
+}

Powered by Google App Engine
This is Rietveld 408576698