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

Issue 10977058: Clean up repo README, add SDK README (Closed)

Created:
8 years, 2 months ago by dgrove
Modified:
8 years, 2 months ago
CC:
reviews_dartlang.org
Visibility:
Public.

Description

Clean up repo README, add SDK README Committed: https://code.google.com/p/dart/source/detail?r=12974

Patch Set 1 #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+32 lines, -10 lines) Patch
M README View 1 chunk +10 lines, -10 lines 2 comments Download
A README.dart-sdk View 1 chunk +22 lines, -0 lines 0 comments Download

Messages

Total messages: 3 (0 generated)
dgrove
8 years, 2 months ago (2012-09-27 16:55:44 UTC) #1
Siggi Cherem (dart-lang)
lgtm https://codereview.chromium.org/10977058/diff/1/README File README (right): https://codereview.chromium.org/10977058/diff/1/README#newcode10 README:10: compiler/ Dart static analyzer. we should hopefully rename ...
8 years, 2 months ago (2012-09-27 17:12:45 UTC) #2
dgrove
8 years, 2 months ago (2012-09-27 17:14:31 UTC) #3
https://codereview.chromium.org/10977058/diff/1/README
File README (right):

https://codereview.chromium.org/10977058/diff/1/README#newcode10
README:10: compiler/      Dart static analyzer.
On 2012/09/27 17:12:45, sigmund wrote:
> we should hopefully rename this to analyzer?

Post-M1, this would make sense.

Powered by Google App Engine
This is Rietveld 408576698