Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code

Issue 29569642: Noissue - Removed update-copyright script (Closed)

Created:
Oct. 9, 2017, 2:10 a.m. by Sebastian Noack
Modified:
Oct. 10, 2017, 5:01 a.m.
Reviewers:
Vasily Kuznetsov
CC:
rosie
Visibility:
Public.

Description

Noissue - Removed update-copyright script

Patch Set 1 #

Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -423 lines) Patch
R update-copyright/README.md View 1 chunk +0 lines, -32 lines 0 comments Download
R update-copyright/tests/data/hg_page.html View 1 chunk +0 lines, -99 lines 0 comments Download
R update-copyright/tests/test_update_copyright.py View 1 chunk +0 lines, -124 lines 0 comments Download
R update-copyright/tox.ini View 1 chunk +0 lines, -18 lines 0 comments Download
R update-copyright/update_copyright.py View 1 chunk +0 lines, -150 lines 0 comments Download

Messages

Total messages: 2
Sebastian Noack
Since we now use "2006-present" instead of updating the final year in the copyright disclaimers, ...
Oct. 9, 2017, 2:16 a.m. (2017-10-09 02:16:55 UTC) #1
Vasily Kuznetsov
Oct. 9, 2017, 8:33 p.m. (2017-10-09 20:33:27 UTC) #2
On 2017/10/09 02:16:55, Sebastian Noack wrote:
> Since we now use "2006-present" instead of updating the final year in the
> copyright disclaimers, we no longer need this script.
> 
> FWIW, when we used this script once, we experienced a bug, causing the push to
> fail due to redundant slashes in the URL. :/
> 
> However, there is no point in fixing this or keep maintaining any code we no
> longer use. So with this patch I'm going to remove this code from the
> repository.

LGTM :'(

Powered by Google App Engine
This is Rietveld