Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(613)

Issue 9981044: Add a linter that checks for copyright headers.

Can't Edit
Can't Publish+Mail
Start Review
Created:
10 years, 10 months ago by benji
Modified:
10 years, 10 months ago
Reviewers:
mp+167536, frankban, gary.poster
Visibility:
Public.

Description

Add a linter that checks for copyright headers. https://code.launchpad.net/~benji/juju-gui/license-notice-linter/+merge/167536 (do not edit description out of merge proposal)

Patch Set 1 #

Patch Set 2 : Add a linter that checks for copyright headers. #

Unified diffs Side-by-side diffs Delta from patch set Stats (+69 lines, -1 line) Patch
M Makefile View 1 1 chunk +13 lines, -1 line 0 comments Download
A [revision details] View 1 1 chunk +2 lines, -0 lines 0 comments Download
M test/test_feature_flags.js View 1 chunk +18 lines, -0 lines 0 comments Download
M test/test_startup.js.top View 1 chunk +18 lines, -0 lines 0 comments Download
M test/test_topology.js View 1 chunk +18 lines, -0 lines 0 comments Download

Messages

Total messages: 4
benji
Please take a look.
10 years, 10 months ago (2013-06-05 13:36:50 UTC) #1
gary.poster
LGTM. Awesome, thank you.
10 years, 10 months ago (2013-06-05 15:51:03 UTC) #2
frankban
LGTM, thank you!
10 years, 10 months ago (2013-06-05 16:27:00 UTC) #3
benji
10 years, 10 months ago (2013-06-05 16:42:51 UTC) #4
*** Submitted:

Add a linter that checks for copyright headers.

R=gary.poster, frankban
CC=
https://codereview.appspot.com/9981044
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld f62528b