Added support for 'description' in pubspec.yaml.

utils/pub/pubspec.dart

 // Copyright (c) 2012, the Dart project authors.  Please see the AUTHORS file
 // for details. All rights reserved. Use of this source code is governed by a
 // BSD-style license that can be found in the LICENSE file.
 
 library pubspec;
 
 import '../../pkg/yaml/lib/yaml.dart';
 import '../../pkg/path/lib/path.dart' as path;
 
 import 'io.dart';
@@ skipping 114 matching lines @@
     // the server:
     // TODO(rnystrom): We should split this validation into separate layers:
     // 1. Stuff that is required in any pubspec to perform any command. Things
     //    like "must have a name". That should go here.
     // 2. Stuff that is required to upload a package. Things like "homepage
     //    must use a valid scheme". That should go elsewhere. pub upload should
     //    call it, and we should provide a separate command to show the user,
     //    and also expose it to the editor in some way.
 
     if (parsedPubspec.containsKey('homepage')) {
-      var homepage = parsedPubspec['homepage'];
-
-      if (homepage is! String) {
-        throw new FormatException(
-            'The "homepage" field should be a string, but was "$homepage".');
-      }
-
-      var goodScheme = new RegExp(r'^https?:');
-      if (!goodScheme.hasMatch(homepage)) {
-        throw new FormatException(
-            'The "homepage" field should be an "http:" or "https:" URL, but '
-            'was "$homepage".');
-      }
+      _validateFieldUrl(parsedPubspec['homepage'], 'homepage');
+    }
+    if (parsedPubspec.containsKey('documentation')) {
+      _validateFieldUrl(parsedPubspec['documentation'], 'documentation');
     }
 
     if (parsedPubspec.containsKey('author') &&
         parsedPubspec['author'] is! String) {
       throw new FormatException(
           'The "author" field should be a string, but was '
           '${parsedPubspec["author"]}.');
     }
 
     if (parsedPubspec.containsKey('authors')) {
@@ skipping 12 matching lines @@
       if (parsedPubspec.containsKey('author')) {
         throw new FormatException('A pubspec should not have both an "author" '
             'and an "authors" field.');
       }
     }
 
     return new Pubspec(name, version, dependencies, environment, parsedPubspec);
   }
 }
 
+/**
+ * Evaluates whether the given [url] for [field] is valid.
+ *
+ * Throws [FormatException] on an invalid url.
+ */
+void _validateFieldUrl(url, String field) {
+  if (url is! String) {
+    throw new FormatException(
+        'The "$field" field should be a string, but was "$url".');
+  }
+
+  var goodScheme = new RegExp(r'^https?:');
+  if (!goodScheme.hasMatch(url)) {
+    throw new FormatException(
+        'The "$field" field should be an "http:" or "https:" URL, but '
+        'was "$url".');
+  }
+}
+
 List<PackageRef> _parseDependencies(SourceRegistry sources, yaml) {
   var dependencies = <PackageRef>[];
 
   // Allow an empty dependencies key.
   if (yaml == null) return dependencies;
 
   if (yaml is! Map || yaml.keys.any((e) => e is! String)) {
     throw new FormatException(
         'The pubspec dependencies should be a map of package names, but '
         'was ${yaml}.');
@@ skipping 45 matching lines @@
 /// The environment-related metadata in the pubspec. Corresponds to the data
 /// under the "environment:" key in the pubspec.
 class PubspecEnvironment {
   /// The version constraint specifying which SDK versions this package works
   /// with.
   final VersionConstraint sdkVersion;
 
   PubspecEnvironment([VersionConstraint sdk])
       : sdkVersion = sdk != null ? sdk : VersionConstraint.any;
 }