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

Issue 1573163002: Update changelog with patch release notes (Closed)

Created:
4 years, 11 months ago by mit
Modified:
4 years, 11 months ago
Reviewers:
Bill Hesse, kevmoo
CC:
reviews_dartlang.org
Base URL:
https://github.com/dart-lang/sdk.git@master
Target Ref:
refs/heads/master
Visibility:
Public.

Description

Kevin/Bill, there was a good suggestion in #25374 that we should maintain the changelog even for SDK patch releases. I think that is a good idea. I went back and added entries for 1.13.1 and 1.13.2. What do you think? Fixes #25374 R=kevmoo@google.com Committed: https://github.com/dart-lang/sdk/commit/1bf4ee0841927f448468ba1216f929b28af31347

Patch Set 1 #

Total comments: 1

Patch Set 2 : Fix long lines #

Patch Set 3 : Rebase #

Unified diffs Side-by-side diffs Delta from patch set Stats (+23 lines, -1 line) Patch
M CHANGELOG.md View 1 2 2 chunks +23 lines, -1 line 0 comments Download

Messages

Total messages: 10 (5 generated)
mit
4 years, 11 months ago (2016-01-11 15:51:12 UTC) #4
mit
4 years, 11 months ago (2016-01-11 15:59:26 UTC) #6
Bill Hesse
https://codereview.chromium.org/1573163002/diff/1/CHANGELOG.md File CHANGELOG.md (right): https://codereview.chromium.org/1573163002/diff/1/CHANGELOG.md#newcode80 CHANGELOG.md:80: * Pub get crash on ARM: Fixes a crash ...
4 years, 11 months ago (2016-01-11 17:55:13 UTC) #7
kevmoo
LGTM – pending long line fix. Thanks!
4 years, 11 months ago (2016-01-11 18:03:01 UTC) #8
mit
4 years, 11 months ago (2016-01-12 10:09:25 UTC) #10
Message was sent while issue was closed.
Committed patchset #3 (id:40001) manually as
1bf4ee0841927f448468ba1216f929b28af31347 (presubmit successful).

Powered by Google App Engine
This is Rietveld 408576698