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

Issue 162763002: Fix crlf to lf-only for third_party/README.chromium.template (Closed)

Created:
6 years, 10 months ago by kochi
Modified:
6 years, 10 months ago
CC:
chromium-reviews
Visibility:
Public.

Description

Fix crlf to lf-only for third_party/README.chromium.template We should use LF-only unless it is really necessary (e.g. .BAT files). Probably this was introduced by crrev.com/250068. BUG=none TBR=mnaganov@chromium.org, darin@chromium.org Committed: https://src.chromium.org/viewvc/chrome?view=rev&revision=250962

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+16 lines, -16 lines) Patch
M third_party/README.chromium.template View 1 chunk +16 lines, -16 lines 0 comments Download

Messages

Total messages: 4 (0 generated)
kochi
Committed patchset #1 manually as r250962 (presubmit successful).
6 years, 10 months ago (2014-02-13 05:31:35 UTC) #1
darin (slow to review)
LGTM
6 years, 10 months ago (2014-02-13 06:55:54 UTC) #2
mnaganov (inactive)
On 2014/02/13 05:31:35, Takayoshi Kochi wrote: > Committed patchset #1 manually as r250962 (presubmit successful). ...
6 years, 10 months ago (2014-02-13 08:23:59 UTC) #3
kochi
6 years, 10 months ago (2014-02-13 08:27:08 UTC) #4
Message was sent while issue was closed.
On 2014/02/13 08:23:59, Mikhail Naganov (Cr) wrote:
> On 2014/02/13 05:31:35, Takayoshi Kochi wrote:
> > Committed patchset #1 manually as r250962 (presubmit successful).
> 
> I apologize. I saw CR characters in the file, but I was supposing, it was like
> that before I started editing it. Thanks for fixing!

No worries.  Thanks for checking.

Powered by Google App Engine
This is Rietveld 408576698