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

Issue 1050283002: Change analyzer version to 0.24.1-alpha.0 in preparation for publishing. (Closed)

Created:
5 years, 8 months ago by Paul Berry
Modified:
5 years, 8 months ago
CC:
reviews_dartlang.org, Bob Nystrom
Target Ref:
refs/remotes/git-svn
Visibility:
Public.

Description

Change analyzer version to 0.24.1-alpha.0 in preparation for publishing. The most recently released version of analyzer that we intend to support is 0.24.0, and there have been no breaking changes since then. Therefore, assuming no further breaking changes occur, the next supported version of analyzer will be 0.24.1. We need to release an unsupported version of analyzer for analysis_server to depend upon. Publishing packages with "dev" in their version string is discouraged (since "dev" means "unpublished"), so we're going to start using "alpha" for this purpose. Therefore, the correct version string for this alpha release is 0.24.1-alpha.0. Note that three versions of analyzer greater than 0.24.1-alpha.0 have already been published: 0.25.0-dev.{1,2,3}. It's not a problem that we're going "backwards" in version number, because (a) those releases contain a "-" in their version string, indicating that they are unsupported, and (b) the minor version should never have been incremented to 25, since there have been no breaking changes since 0.24. R=brianwilkerson@google.com, rnystrom@google.com Committed: https://code.google.com/p/dart/source/detail?r=44868

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+1 line, -1 line) Patch
M pkg/analyzer/pubspec.yaml View 1 chunk +1 line, -1 line 0 comments Download

Messages

Total messages: 6 (2 generated)
Paul Berry
Once this CL has landed, I will publish 0.24.1-alpha.0 to pub, and then I will ...
5 years, 8 months ago (2015-04-01 20:23:29 UTC) #2
Brian Wilkerson
LGTM
5 years, 8 months ago (2015-04-01 20:31:47 UTC) #3
Bob Nystrom
LGTM! Your explanation sounds spot-on to me. - bob
5 years, 8 months ago (2015-04-01 21:12:36 UTC) #5
Paul Berry
5 years, 8 months ago (2015-04-02 20:01:53 UTC) #6
Message was sent while issue was closed.
Committed patchset #1 (id:1) manually as 44868 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698