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

Issue 12545045: http: clear out clirr-ignored-differences (Closed)

Can't Edit
Can't Publish+Mail
Start Review
Created:
12 years ago by yanivi
Modified:
12 years ago
Reviewers:
ngmiceli
Base URL:
https://code.google.com/p/google-http-java-client/
Visibility:
Public.

Description

http: clear out clirr-ignored-differences

Patch Set 1 : fix patch #

Total comments: 2
Unified diffs Side-by-side diffs Delta from patch set Stats (+0 lines, -10 lines) Patch
M clirr-ignored-differences.xml View 1 chunk +0 lines, -10 lines 2 comments Download

Messages

Total messages: 6
yanivi
12 years ago (2013-08-09 14:09:25 UTC) #1
ngmiceli
https://codereview.appspot.com/12545045/diff/5001/clirr-ignored-differences.xml File clirr-ignored-differences.xml (left): https://codereview.appspot.com/12545045/diff/5001/clirr-ignored-differences.xml#oldcode17 clirr-ignored-differences.xml:17: <className>com/google/api/client/repackaged/**</className> I don't believe we need this anymore either. ...
12 years ago (2013-08-09 14:15:21 UTC) #2
yanivi
https://codereview.appspot.com/12545045/diff/5001/clirr-ignored-differences.xml File clirr-ignored-differences.xml (left): https://codereview.appspot.com/12545045/diff/5001/clirr-ignored-differences.xml#oldcode17 clirr-ignored-differences.xml:17: <className>com/google/api/client/repackaged/**</className> On 2013/08/09 14:15:21, ngmiceli wrote: > I don't ...
12 years ago (2013-08-09 14:50:57 UTC) #3
ngmiceli
mvn clirr:check succeeds for me with a fully-empty clirr-ignored-differences file. Does it not for you?
12 years ago (2013-08-12 16:56:55 UTC) #4
yanivi
mvn -q clirr:check [ERROR] com.google.api.client.repackaged.com.google.common.annotations.Beta: Class com.google.api.client.repackaged.com.google.common.annotations.Beta removed [ERROR] com.google.api.client.repackaged.com.google.common.annotations.GwtCompatible: Class com.google.api.client.repackaged.com.google.common.annotations.GwtCompatible removed [ERROR] com.google.api.client.repackaged.com.google.common.annotations.VisibleForTesting: ...
12 years ago (2013-08-12 17:02:07 UTC) #5
ngmiceli
12 years ago (2013-08-12 17:07:48 UTC) #6
LGTM
Sign in to reply to this message.

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