Bump to 0.2.1.

tool/grind.dart

 // Copyright (c) 2015, 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.
 
 import 'package:grinder/grinder.dart';
+import 'package:pub_semver/pub_semver.dart';
 import 'package:yaml/yaml.dart' as yaml;
 
+/// Matches the version line in dart_style's pubspec.
+final versionPattern = new RegExp(r"^version: .*$", multiLine: true);

[jakemac, 2015/12/07 22:25:17]

I would typically make this private even in entry point scripts, but up to you
:)

[Bob Nystrom, 2015/12/07 22:50:35]

Done. I don't usually bother for entry points, but it doesn't hurt to make it
private.

+
 main(args) => grind(args);
 
 @DefaultTask()
 @Task()
 validate() async {
+  // Test it.
   await new TestRunner().testAsync();
+
+  // Make sure it's warning clean.
   Analyzer.analyze("bin/format.dart", fatalWarnings: true);
-  DartFmt.format(".");
+
+  // Format it.
+  Dart.run("bin/format.dart", arguments: ["-w", "."]);
 }
 
 /// Gets ready to publish a new version of the package.
 ///
 /// To publish a version, you need to:
 ///
-///   1. Bump the version in the pubspec to a non-dev number.
+///   1. Make sure the version in the pubspec is a "-dev" number. This should
+///      already be the case since you've already landed patches that change
+///      the formatter and bumped to that as a consequence.
 ///
 ///   2. Run this task:
 ///
 ///         pub run grinder bump
 ///
 ///   3. Commit the change to a branch.
 ///
 ///   4. Send it out for review:
 ///
 ///         git cl upload
 ///
 ///   5. After the review is complete, land it:
 ///
 ///         git cl land
 ///
 ///   6. Tag the commit:
 ///
 ///         git tag -a "<version>" -m "<version>"
 ///         git push origin <version>
 ///
 ///   7. Publish the package:
 ///
 ///         pub lish
 @Task()
 @Depends(validate)
 bump() async {
   // Read the version from the pubspec.
-  var pubspec = yaml.loadYaml(getFile("pubspec.yaml").readAsStringSync());
-  var version = pubspec["version"];
-  print(version);
+  var pubspecFile = getFile("pubspec.yaml");
+  var pubspec = pubspecFile.readAsStringSync();
+  var version = new Version.parse(yaml.loadYaml(pubspec)["version"]);
 
-  if (version.contains("-dev")) throw "Cannot publish a dev version.";
+  // Require a "-dev" version since we don't otherwise know what to bump it to.
+  if (!version.isPreRelease) throw "Cannot publish non-dev version $version.";
+
+  // Don't allow versions like "1.2.3-dev+4" because it's not clear if the
+  // user intended the "+4" to be discarded or not.
+  if (version.build.isNotEmpty) throw "Cannot publish build version $version.";
+
+  var bumped = new Version(version.major, version.minor, version.patch);
+
+  // Update the version in the pubspec.
+  pubspec = pubspec.replaceAll(versionPattern, "version: $bumped");

[jakemac, 2015/12/07 22:25:17]

If the Version class has an explicit getter for the valid version string I would
use it, otherwise this is probably fine.

[Bob Nystrom, 2015/12/07 22:50:35]

I want to not just match the version string in case some dependency happens to
coincidentally have the same version as dart_style.

[jakemac, 2015/12/07 23:02:53]

Oh, I wasn't very clear here :). I was just talking about relying on the default
toString() of Version (via `$bumped`). I don't see an explicit getter on the
Version class which gives the formatted version though anyways, so I think you
are fine.

+  pubspecFile.writeAsStringSync(pubspec);
 
   // Update the version constant in bin/format.dart.
   var binFormatFile = getFile("bin/format.dart");
   var binFormat = binFormatFile.readAsStringSync().replaceAll(
-      new RegExp(r'const version = "[^"]+";'), 'const version = "$version";');
+      new RegExp(r'const version = "[^"]+";'), 'const version = "$bumped";');
   binFormatFile.writeAsStringSync(binFormat);
-  print("Updated version constant to '$version'.");
+  log("Updated version to '$bumped'.");
 }