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

Issues Search
    My Issues | Starred     Open | Closed | All

Issue 1718123003: Add header text to proto-gae tool (Closed)

Created:
4 years, 10 months ago by iannucci
Modified:
4 years, 10 months ago
Reviewers:
dnj, estaab, martiniss, hinoka
CC:
chromium-reviews, infra-reviews+luci-gae_chromium.org
Base URL:
https://github.com/luci/gae.git@master
Target Ref:
refs/heads/master
Project:
luci-gae
Visibility:
Public.

Description

Add header text to proto-gae tool In order to appease our presubmit scripts, default to the Chromium Authors copyright header. R=dnj@chromium.org, estaab@chromium.org, hinoka@chromium.org, martiniss@chromium.org BUG= Committed: https://github.com/luci/gae/commit/24f10074060aae9fef22a70b7ca764da668052ee

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+11 lines, -1 line) Patch
M tools/proto-gae/proto_gae.go View 5 chunks +11 lines, -1 line 0 comments Download

Messages

Total messages: 7 (2 generated)
iannucci
4 years, 10 months ago (2016-02-22 22:44:34 UTC) #1
iannucci
Super short, PTAL
4 years, 10 months ago (2016-02-22 22:45:07 UTC) #2
martiniss
lgtm copy all the rights
4 years, 10 months ago (2016-02-22 23:45:46 UTC) #3
commit-bot: I haz the power
CQ is trying da patch. Follow status at https://chromium-cq-status.appspot.com/patch-status/1718123003/1 View timeline at https://chromium-cq-status.appspot.com/patch-timeline/1718123003/1
4 years, 10 months ago (2016-02-22 23:56:30 UTC) #5
commit-bot: I haz the power
4 years, 10 months ago (2016-02-22 23:59:04 UTC) #7
Message was sent while issue was closed. 
Committed patchset #1 (id:1) as
https://github.com/luci/gae/commit/24f10074060aae9fef22a70b7ca764da668052ee

Powered by Google App Engine
This is Rietveld 408576698