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

Issue 2599943002: Exclude c/b/r/options/ and c/b/r/history from clang-format (Closed)

Created:
4 years ago by Dan Beam
Modified:
3 years, 11 months ago
Reviewers:
tsergeant
CC:
chromium-reviews, dbeam+watch-options_chromium.org, michaelpg+watch-options_chromium.org, Patrick Dubroy, dbeam+watch-history_chromium.org, pam+watch_chromium.org, arv+watch_chromium.org
Target Ref:
refs/pending/heads/master
Project:
chromium
Visibility:
Public.

Description

Exclude c/b/r/options/ and c/b/r/history from clang-format Both codebases are deprecated and would only temporarily benefit for more risk than it's worth. R=tsergeant@chromium.org BUG=567770 NOTRY=true

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+4 lines, -0 lines) Patch
A chrome/browser/resources/history/.clang-format View 1 chunk +2 lines, -0 lines 0 comments Download
A chrome/browser/resources/options/.clang-format View 1 chunk +2 lines, -0 lines 0 comments Download

Messages

Total messages: 9 (6 generated)
Dan Beam
4 years ago (2016-12-23 02:02:48 UTC) #2
tsergeant
lgtm
3 years, 12 months ago (2016-12-27 23:44:42 UTC) #8
Dan Beam
3 years, 11 months ago (2016-12-28 18:06:01 UTC) #9
btw, folks on chromium-dev@ want less .clang-format files in the codebase so i
may just add a blacklist in c/b/r/PRESUBMIT.py instead

Powered by Google App Engine
This is Rietveld 408576698