danger-android_lint 0.0.8 → 0.0.11

checksums.yaml

@@ -1,7 +1,7 @@
 ---
-SHA1:
-  metadata.gz: 6fb3bbc5b97d33a9495f3c3811b6a3dbe6932dcc
-  data.tar.gz: 2c2d08a45644d86fbd60364fc80ec2bbfbe8eac9
+SHA256:
+  metadata.gz: 79e5bfcae51d8156567b963bb7ac48ea67e0db8f2465c99746ffad9e922d4b12
+  data.tar.gz: b9e3527bba12cbd1d88713a6bd33ced049ad8edf630c2e5820eb8065fbe63267
 SHA512:
-  metadata.gz: 6fb633fd0c8c48391c4097103e0779ca0d8eff4d6bfcb6824c9400f1a0911c00c5b06d4c122fefcb6eb59745e7155411d5f77f5844ef17103835a61a9465f94c
-  data.tar.gz: 4a10c53f60ebb6db842c88e08c3121bd42c95811f0c639578c73dbc12e7e32cab1763ba24e3b007ffc23d6651038c206af76ddeb9d07c140dbd2bf8faf23faea
+  metadata.gz: 1f8811a723a32f71f8e505259f0df20b737cf7e8aa545e0b825ac8840f4411214be825d195f0ba40baad8b35f91ab6aa153ba915deaafdc2b9aa732ff57b4ff3
+  data.tar.gz: 43e252db44f9d9254315b70c640c37edc4f8ee2996f279ab3afb4131a5ed371fa1734887afe776f48f4fabc39f3979e7127e2de38d18034e2009f4335276c004

data/.circleci/config.yml

@@ -0,0 +1,13 @@
+version: 2.1
+orbs:
+  ruby: circleci/ruby@1.0
+
+jobs:
+  build:
+    docker:
+      - image: circleci/ruby:2.7
+    steps:
+      - checkout
+      - run: gem install bundler:2.2.10
+      - run: bundle install
+      - run: bundle exec rake spec

data/.github/CODEOWNERS

@@ -1 +1 @@
-*	@loadsmart/mobile
+*	@barbosa

data/.ruby-version

@@ -1 +1 @@
-2.3.5
+3.0.2

data/CHANGELOG.md

@@ -1,3 +1,14 @@
+# 0.0.11
+
+- Add `excluding_issue_ids` parameter to skip showing issues with the given IDs, e.g. "MissingTranslation" ([@petitJAM](https://github.com/petitJAM))
+
+# 0.0.10
+
+- Fix `filtering_lines` parameter to also apply to when `inline_mode` is `false` ([@petitJAM](https://github.com/petitJAM))
+
+# 0.0.9
+- Add `filtering_lines` parameter, to show lint issues only on modified lines ([@ShivamPokhriyal](https://github.com/ShivamPokhriyal))
+
 # 0.0.8
 - Fix security issues ([@barbosa](https://github.com/barbosa))
 - Expose report message ([@adamstrange])(https://github.com/adamstrange))

data/Gemfile.lock

@@ -1,50 +1,72 @@
 PATH
   remote: .
   specs:
-    danger-android_lint (0.0.6)
+    danger-android_lint (0.0.11)
       danger-plugin-api (~> 1.0)
       oga
 
 GEM
   remote: https://rubygems.org/
   specs:
-    addressable (2.7.0)
+    addressable (2.8.0)
       public_suffix (>= 2.0.2, < 5.0)
     ansi (1.5.0)
-    ast (2.4.0)
-    claide (1.0.3)
+    ast (2.4.2)
+    claide (1.1.0)
     claide-plugins (0.9.2)
       cork
       nap
       open4 (~> 1.3)
-    coderay (1.1.2)
+    coderay (1.1.3)
     colored2 (3.1.2)
     cork (0.3.0)
       colored2 (~> 3.1)
-    danger (6.1.0)
+    danger (8.5.0)
       claide (~> 1.0)
       claide-plugins (>= 0.9.2)
       colored2 (~> 3.1)
       cork (~> 0.1)
-      faraday (~> 0.9)
+      faraday (>= 0.9.0, < 2.0)
       faraday-http-cache (~> 2.0)
-      git (~> 1.5)
-      kramdown (~> 2.0)
+      git (~> 1.7)
+      kramdown (~> 2.3)
       kramdown-parser-gfm (~> 1.0)
       no_proxy_fix
       octokit (~> 4.7)
-      terminal-table (~> 1)
+      terminal-table (>= 1, < 4)
     danger-plugin-api (1.0.0)
       danger (> 2.0)
-    diff-lcs (1.3)
-    faraday (0.17.1)
+    diff-lcs (1.4.4)
+    faraday (1.10.0)
+      faraday-em_http (~> 1.0)
+      faraday-em_synchrony (~> 1.0)
+      faraday-excon (~> 1.1)
+      faraday-httpclient (~> 1.0)
+      faraday-multipart (~> 1.0)
+      faraday-net_http (~> 1.0)
+      faraday-net_http_persistent (~> 1.0)
+      faraday-patron (~> 1.0)
+      faraday-rack (~> 1.0)
+      faraday-retry (~> 1.0)
+      ruby2_keywords (>= 0.0.4)
+    faraday-em_http (1.0.0)
+    faraday-em_synchrony (1.0.0)
+    faraday-excon (1.1.0)
+    faraday-http-cache (2.2.0)
+      faraday (>= 0.8)
+    faraday-httpclient (1.0.1)
+    faraday-multipart (1.0.3)
       multipart-post (>= 1.2, < 3)
-    faraday-http-cache (2.0.0)
-      faraday (~> 0.8)
-    ffi (1.11.3)
-    formatador (0.2.5)
-    git (1.5.0)
-    guard (2.16.1)
+    faraday-net_http (1.0.1)
+    faraday-net_http_persistent (1.2.0)
+    faraday-patron (1.0.0)
+    faraday-rack (1.0.0)
+    faraday-retry (1.0.3)
+    ffi (1.15.3)
+    formatador (0.3.0)
+    git (1.10.2)
+      rchardet (~> 1.8)
+    guard (2.17.0)
       formatador (>= 0.2.4)
       listen (>= 2.7, < 4.0)
       lumberjack (>= 1.0.12, < 2.0)
@@ -58,14 +80,15 @@ GEM
       guard (~> 2.1)
       guard-compat (~> 1.1)
       rspec (>= 2.99.0, < 4.0)
-    kramdown (2.1.0)
+    kramdown (2.3.2)
+      rexml
     kramdown-parser-gfm (1.1.0)
       kramdown (~> 2.0)
     listen (3.0.7)
       rb-fsevent (>= 0.9.3)
       rb-inotify (>= 0.9.7)
-    lumberjack (1.0.13)
-    method_source (0.9.2)
+    lumberjack (1.2.8)
+    method_source (1.0.0)
     multipart-post (2.1.1)
     nap (1.1.0)
     nenv (0.3.0)
@@ -73,39 +96,42 @@ GEM
     notiffany (0.1.3)
       nenv (~> 0.1)
       shellany (~> 0.0)
-    octokit (4.14.0)
+    octokit (4.22.0)
+      faraday (>= 0.9)
       sawyer (~> 0.8.0, >= 0.5.3)
-    oga (3.0)
+    oga (3.3)
       ast
       ruby-ll (~> 2.1)
     open4 (1.3.4)
-    parallel (1.19.1)
-    parser (2.6.5.0)
-      ast (~> 2.4.0)
-    powerpack (0.1.2)
-    pry (0.12.2)
-      coderay (~> 1.1.0)
-      method_source (~> 0.9.0)
-    public_suffix (4.0.1)
+    parallel (1.20.1)
+    parser (2.7.2.0)
+      ast (~> 2.4.1)
+    powerpack (0.1.3)
+    pry (0.14.1)
+      coderay (~> 1.1)
+      method_source (~> 1.0)
+    public_suffix (4.0.6)
     rainbow (2.2.2)
       rake
-    rake (10.5.0)
-    rb-fsevent (0.10.3)
+    rake (13.0.3)
+    rb-fsevent (0.11.0)
     rb-inotify (0.10.1)
       ffi (~> 1.0)
-    rspec (3.9.0)
-      rspec-core (~> 3.9.0)
-      rspec-expectations (~> 3.9.0)
-      rspec-mocks (~> 3.9.0)
-    rspec-core (3.9.0)
-      rspec-support (~> 3.9.0)
-    rspec-expectations (3.9.0)
+    rchardet (1.8.0)
+    rexml (3.2.5)
+    rspec (3.10.0)
+      rspec-core (~> 3.10.0)
+      rspec-expectations (~> 3.10.0)
+      rspec-mocks (~> 3.10.0)
+    rspec-core (3.10.1)
+      rspec-support (~> 3.10.0)
+    rspec-expectations (3.10.1)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-mocks (3.9.0)
+      rspec-support (~> 3.10.0)
+    rspec-mocks (3.10.2)
       diff-lcs (>= 1.2.0, < 2.0)
-      rspec-support (~> 3.9.0)
-    rspec-support (3.9.0)
+      rspec-support (~> 3.10.0)
+    rspec-support (3.10.2)
     rubocop (0.49.1)
       parallel (~> 1.10)
       parser (>= 2.3.3.1, < 3.0)
@@ -116,31 +142,34 @@ GEM
     ruby-ll (2.1.2)
       ansi
       ast
-    ruby-progressbar (1.10.1)
+    ruby-progressbar (1.11.0)
+    ruby2_keywords (0.0.5)
     sawyer (0.8.2)
       addressable (>= 2.3.5)
       faraday (> 0.8, < 2.0)
     shellany (0.0.1)
-    terminal-table (1.8.0)
-      unicode-display_width (~> 1.1, >= 1.1.1)
-    thor (1.0.1)
-    unicode-display_width (1.6.0)
-    yard (0.9.20)
+    terminal-table (3.0.2)
+      unicode-display_width (>= 1.1.1, < 3)
+    thor (1.1.0)
+    unicode-display_width (1.7.0)
+    yard (0.9.26)
 
 PLATFORMS
-  ruby
+  universal-darwin-20
+  x86_64-darwin-21
+  x86_64-linux
 
 DEPENDENCIES
-  bundler (~> 1.3)
+  bundler (>= 2.2.10)
   danger-android_lint!
   guard (~> 2.14)
   guard-rspec (~> 4.7)
   listen (= 3.0.7)
   pry
-  rake (~> 10.0)
+  rake (~> 13.0)
   rspec (~> 3.4)
   rubocop (~> 0.49.0)
   yard (~> 0.9.11)
 
 BUNDLED WITH
-   1.17.3
+   2.2.22

data/README.md

@@ -85,6 +85,24 @@ android_lint.filtering = true
 android_lint.lint
 ```
 
+Additionally, you can further filter to only the modified lines by setting the `filtering_lines` parameter to `true`.
+
+```rb
+android_lint.filtering_lines = true
+android_lint.lint
+```
+
+#### Filter issues by ID
+
+In some cases you may want to conditionally ignore specific lint warnings without fully disabling
+them in your `lintOptions` configuration. To do that, you can pass a list of IDs to the
+`excluding_issue_ids` parameter.
+
+```rb
+android_lint.excluding_issue_ids = ["MissingTranslation"]
+android_lint.lint
+```
+
 #### Make Danger comment directly on the line instead of printing a Markdown table (GitHub only)
 
 ```rb

data/Rakefile

@@ -8,7 +8,7 @@ task default: :specs
 
 task :spec do
   Rake::Task['specs'].invoke
-  Rake::Task['rubocop'].invoke
+  # Rake::Task['rubocop'].invoke
   Rake::Task['spec_docs'].invoke
 end
 

data/atlantis.yaml

@@ -0,0 +1,5 @@
+version: 3
+projects:
+  - dir: .
+    autoplan:
+      enabled: false

data/danger-android_lint.gemspec

@@ -23,8 +23,8 @@ Gem::Specification.new do |spec|
   spec.add_runtime_dependency 'danger-plugin-api', '~> 1.0'
 
   # General ruby development
-  spec.add_development_dependency 'bundler', '~> 1.3'
-  spec.add_development_dependency 'rake', '~> 10.0'
+  spec.add_development_dependency 'bundler', '>= 2.2.10'
+  spec.add_development_dependency 'rake', '~> 13.0'
 
   # Testing support
   spec.add_development_dependency 'rspec', '~> 3.4'

data/lib/android_lint/gem_version.rb

@@ -1,3 +1,3 @@
 module AndroidLint
-  VERSION = "0.0.8".freeze
+  VERSION = "0.0.11".freeze
 end

data/lib/android_lint/plugin.rb

@@ -84,6 +84,12 @@ module Danger
     # Only show messages within changed files.
     attr_accessor :filtering
 
+    # Only show messages for the modified lines.
+    attr_accessor :filtering_lines
+
+    # Only show messages for issues not in this list.
+    attr_accessor :excluding_issue_ids
+
     # Calls lint task of your gradle project.
     # It fails if `gradlew` cannot be found inside current directory.
     # It fails if `severity` level is not a valid option.
@@ -113,7 +119,7 @@ module Danger
       filtered_issues = filter_issues_by_severity(issues)
 
       message = ""
-      
+
       if inline_mode
         # Report with inline comment
         send_inline_comment(filtered_issues)
@@ -121,7 +127,7 @@ module Danger
         message = message_for_issues(filtered_issues)
         markdown("### AndroidLint found issues\n\n" + message) unless message.to_s.empty?
       end
-      
+
       message
     end
 
@@ -164,11 +170,17 @@ module Danger
       message = ""
 
       results.each do |r|
+        issue_id = r.get('id')
+        next if excluding_issue_ids && excluding_issue_ids.include?(issue_id)
         location = r.xpath('location').first
         filename = location.get('file').gsub(dir, "")
-        next unless !filtering || (target_files.include? filename)
-        line = location.get('line') || 'N/A'
+        next unless (!filtering && !filtering_lines) || (target_files.include? filename)
+        line = location.get('line').to_i || 'N/A'
         reason = r.get('message')
+        if filtering_lines
+          added_lines = parse_added_line_numbers(git.diff[filename].patch)
+          next unless added_lines.include? line
+        end
         count = count + 1
         message << "`#{filename}` | #{line} | #{reason} \n"
       end
@@ -186,7 +198,7 @@ module Danger
     # Send inline comment with danger's warn or fail method
     #
     # @return [void]
-    def send_inline_comment (issues)
+    def send_inline_comment(issues)
       target_files = (git.modified_files - git.deleted_files) + git.added_files
       dir = "#{Dir.pwd}/"
       SEVERITY_LEVELS.reverse.each do |level|
@@ -195,13 +207,42 @@ module Danger
         filtered.each do |r|
           location = r.xpath('location').first
           filename = location.get('file').gsub(dir, "")
-          next unless !filtering || (target_files.include? filename)
+          next unless (!filtering && !filtering_lines) || (target_files.include? filename)
           line = (location.get('line') || "0").to_i
+          if filtering_lines
+            added_lines = parse_added_line_numbers(git.diff[filename].patch)
+            next unless added_lines.include? line
+          end
           send(level === "Warning" ? "warn" : "fail", r.get('message'), file: filename, line: line)
         end
       end
     end
 
+    # Parses git diff of a file and retuns an array of added line numbers.
+    def parse_added_line_numbers(diff)
+      current_line_number = nil
+      added_line_numbers = []
+      diff_lines = diff.strip.split("\n")
+      diff_lines.each_with_index do |line, index|
+        if m = /\+(\d+)(?:,\d+)? @@/.match(line)
+          # (e.g. @@ -32,10 +32,7 @@)
+          current_line_number = Integer(m[1])
+        else
+          if !current_line_number.nil?
+            if line.start_with?('+')
+              # added line
+              added_line_numbers.push current_line_number
+              current_line_number += 1
+            elsif !line.start_with?('-')
+              # unmodified line
+              current_line_number += 1
+            end
+          end
+        end
+      end
+      added_line_numbers
+    end
+
     def gradlew_exists?
       `ls gradlew`.strip.empty? == false
     end

data/spec/android_lint_spec.rb

@@ -146,7 +146,7 @@ module Danger
           expect(markdown).to be_nil
         end
 
-        it 'Doesn`t print anything if no errors were found' do
+        it 'Doesn`t print anything if no errors were found at the set minimum severity level' do
           fake_result = File.open("spec/fixtures/lint-result-without-fatal.xml")
           allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
 
@@ -188,8 +188,96 @@ module Danger
           expect(warn).not_to include("Implicitly using the default locale is a common source of bugs: Use `String.format(Locale, ...)` instead")
         end
 
-      end
+        describe 'excluding_issue_ids' do
+          before do
+            fake_result = File.open("spec/fixtures/lint-result-with-everything.xml")
+            allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
+          end
+
+          it 'Does not print ignored issues' do
+            @android_lint.excluding_issue_ids = ["MissingTranslation", "RtlEnabled"]
+            @android_lint.lint
+
+            markdown = @android_lint.status_report[:markdowns].first.message
+            expect(markdown).to include("Implicitly using the default locale")
+            expect(markdown).not_to include("is not translated in")
+            expect(markdown).not_to include("The project references RTL attributes, but does not explicitly enable or disable RTL support with `android:supportsRtl` in the manifest")
+          end
+        end
 
+        describe "for a modified file" do
+          before do
+            allow(Dir).to receive(:pwd).and_return("/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android")
+
+            allow(@android_lint.git).to receive(:modified_files).and_return([
+              "app/build.gradle",
+            ])
+
+            fake_patch = File.read("spec/fixtures/pr-diff.diff")
+            diff = [OpenStruct.new(type: "modified", path: "app/build.gradle")]
+            allow(@android_lint.git).to receive(:diff).and_return(diff)
+            diff_for_file = OpenStruct.new(insertions: 3)
+            allow(diff).to receive(:[]).with("app/build.gradle").and_return(diff_for_file)
+            allow(diff_for_file).to receive(:patch).and_return(fake_patch)
+
+            fake_result = File.open("spec/fixtures/lint-result-for-pr.xml")
+            allow(File).to receive(:open).with(@android_lint.report_file).and_return(fake_result)
+          end
+
+          context 'when inline_mode: true' do
+            it 'with filtering_lines, only show issues in modified lines' do
+              @android_lint.filtering_lines = true
+              @android_lint.lint inline_mode: true
+
+              error = @android_lint.status_report[:errors]
+              expect(error).to include("fake message three")
+
+              expect(error).not_to include("fake message one")
+              expect(error).not_to include("fake message two")
+              expect(error).not_to include("fake message in unmodified file")
+            end
+
+            it 'with filtering, show all issues in modified files' do
+              @android_lint.filtering = true
+              @android_lint.lint inline_mode: true
+
+              error = @android_lint.status_report[:errors]
+              expect(error).to include("fake message one")
+              expect(error).to include("fake message two")
+              expect(error).to include("fake message three")
+
+              expect(error).not_to include("fake message in unmodified file")
+            end
+          end
+
+          context 'when inline_mode: false' do
+            it 'with filtering_lines, only show issues in modified lines' do
+              @android_lint.filtering_lines = true
+              @android_lint.lint inline_mode: false
+
+              puts @android_lint.status_report[:markdowns]
+              message = @android_lint.status_report[:markdowns][0].to_s
+              expect(message).to include("fake message three")
+
+              expect(message).not_to include("fake message one")
+              expect(message).not_to include("fake message two")
+              expect(message).not_to include("fake message in unmodified file")
+            end
+
+            it 'with filtering, show all issues in modified files' do
+              @android_lint.filtering = true
+              @android_lint.lint inline_mode: false
+
+              message = @android_lint.status_report[:markdowns][0].to_s
+              expect(message).to include("fake message one")
+              expect(message).to include("fake message two")
+              expect(message).to include("fake message three")
+
+              expect(message).not_to include("fake message in unmodified file")
+            end
+          end
+        end
+      end
     end
   end
 end

data/spec/fixtures/lint-result-for-pr.xml

@@ -0,0 +1,62 @@
+<?xml version="1.0" encoding="UTF-8"?>
+<issues format="5" by="lint 4.0.0">
+    <issue
+        id="DuplicatePlatformClasses"
+        severity="Error"
+        message="fake message one"
+        category="Correctness"
+        priority="8"
+        summary="Duplicate Platform Classes"
+        explanation="There are a number of libraries that duplicate not just functionality of the Android platform but using the exact same class names as the ones provided in Android -- for example the apache http classes. This can lead to unexpected crashes.&#xA;&#xA;To solve this, you need to either find a newer version of the library which no longer has this problem, or to repackage the library (and all of its dependencies) using something like the `jarjar` tool, or finally, rewriting the code to use different APIs (for example, for http code, consider using `HttpUrlConnection` or a library like `okhttp`).">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"/>
+    </issue>
+
+    <issue
+        id="DuplicatePlatformClasses"
+        severity="Error"
+        message="fake message two"
+        category="Correctness"
+        priority="8"
+        summary="Duplicate Platform Classes"
+        explanation="There are a number of libraries that duplicate not just functionality of the Android platform but using the exact same class names as the ones provided in Android -- for example the apache http classes. This can lead to unexpected crashes.&#xA;&#xA;To solve this, you need to either find a newer version of the library which no longer has this problem, or to repackage the library (and all of its dependencies) using something like the `jarjar` tool, or finally, rewriting the code to use different APIs (for example, for http code, consider using `HttpUrlConnection` or a library like `okhttp`)."
+        errorLine1="    testImplementation &apos;org.json:json:20140107&apos;"
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"
+            line="52"
+            column="25"/>
+    </issue>
+
+    <issue
+        id="FakeId"
+        severity="Error"
+        message="fake message three"
+        category="Correctness"
+        priority="8"
+        summary="Fake summary"
+        explanation="Fake explanation."
+        errorLine1="// debugImplementation because LeakCanary should only run in debug builds."
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/build.gradle"
+            line="36"
+            column="10"/>
+    </issue>
+
+    <issue
+        id="2"
+        severity="Error"
+        message="fake message in unmodified file"
+        category="Correctness"
+        priority="8"
+        summary="Fake summary"
+        explanation="Fake explanation."
+        errorLine1="// debugImplementation because LeakCanary should only run in debug builds."
+        errorLine2="                        ~~~~~~~~~~~~~">
+        <location
+            file="/Users/shivampokhriyal/Documents/projects/Commcare/commcare-android/app/AndroidManifest.xml"
+            line="36"
+            column="10"/>
+    </issue>
+</issues>

data/spec/fixtures/lint-result-with-everything.xml

@@ -61,4 +61,34 @@ If you want the methods to just perform ASCII replacement, for example to conver
             column="16"/>
     </issue>
 
+    <issue
+        id="MissingTranslation"
+        severity="Error"
+        message="&quot;someStringResource&quot; is not translated in &quot;de&quot; (German), &quot;fi&quot; (Finnish), &quot;ru&quot; (Russian), &quot;sv&quot; (Swedish), &q
+        uot;ko&quot; (Korean), &quot;pt&quot; (Portuguese), &quot;el&quot; (Greek), &quot;en&quot; (English), &quot;it&quot; (Italian), &quot;fr&quot; (French), &quot;es&quot; (Spanish), &quot;cs&q
+        uot; (Czech), &quot;nb&quot; (Norwegian Bokmål), &quot;ja&quot; (Japanese), &quot;da&quot; (Danish), &quot;ca&quot; (Catalan), &quot;nl&quot; (Dutch)"
+        category="Correctness:Messages"
+        priority="8"
+        summary="Incomplete translation"
+        explanation="If an application has more than one locale, then all the strings declared in one language should also be translated in all other languages.&#xA;&#xA;If the string should **not** be translated, you can add the attribute `translatable=&quot;false&quot;` on the `&lt;string>` element, or you can define all your non-translatable strings in a resource file called `donottranslate.xml`. Or, you can ignore the issue with a `tools:ignore=&quot;MissingTranslation&quot;` attribute.&#xA;&#xA;You can tell lint (and other tools) which language is the default language in your `res/values/` folder by specifying `tools:locale=&quot;languageCode&quot;` for the root `&lt;resources>` element in your resource file. (The `tools` prefix refers to the namespace declaration `http://schemas.android.com/tools`.)"
+        errorLine1="    &lt;string name=&quot;someStringResource&quot;>Hello this is a string resource.&lt;/string>"
+        errorLine2="            ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~">
+        <location
+            file="/Users/gustavo/Developer/app-android/app/src/main/res/values/strings.xml"
+            line="10"
+            column="13"/>
+    </issue>
+
+    <issue
+        id="RtlEnabled"
+        severity="Warning"
+        message="The project references RTL attributes, but does not explicitly enable or disable RTL support with `android:supportsRtl` in the manifest"
+        category="Internationalization:Bidirectional Text"
+        priority="3"
+        summary="Using RTL attributes without enabling RTL support"
+        explanation="To enable right-to-left support, when running on API 17 and higher, you must set the `android:supportsRtl` attribute in the manifest `&lt;application>` element.&#xA;&#xA;If you have started adding RTL attributes, but have not yet finished the migration, you can set the attribute to false to satisfy this lint check.">
+        <location
+            file="/Users/gustavo/Developer/app-android/app/src/main/AndroidManifest.xml"/>
+    </issue>
+
 </issues>

data/spec/fixtures/pr-diff.diff

@@ -0,0 +1,14 @@
+diff --git a/app/build.gradle b/app/build.gradle
+index 6dcc647b8..a1365bd75 100644
+--- a/app/build.gradle
++++ b/app/build.gradle
+@@ -33,6 +33,9 @@ configurations {
+ }
+
+ dependencies {
++    // debugImplementation because LeakCanary should only run in debug builds.
++    debugImplementation 'com.squareup.leakcanary:leakcanary-android:2.7'
++
+     testImplementation 'junit:junit:4.12'
+     testImplementation('org.robolectric:robolectric:4.3.1') {
+         exclude(group: 'org.bouncycastle', module: 'bcprov-jdk15on')

metadata

@@ -1,14 +1,14 @@
 --- !ruby/object:Gem::Specification
 name: danger-android_lint
 version: !ruby/object:Gem::Version
-  version: 0.0.8
+  version: 0.0.11
 platform: ruby
 authors:
 - Gustavo Barbosa
-autorequire: 
+autorequire:
 bindir: bin
 cert_chain: []
-date: 2019-12-24 00:00:00.000000000 Z
+date: 2022-07-11 00:00:00.000000000 Z
 dependencies:
 - !ruby/object:Gem::Dependency
   name: oga
@@ -42,30 +42,30 @@ dependencies:
   name: bundler
   requirement: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: 2.2.10
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
-    - - "~>"
+    - - ">="
       - !ruby/object:Gem::Version
-        version: '1.3'
+        version: 2.2.10
 - !ruby/object:Gem::Dependency
   name: rake
   requirement: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
   type: :development
   prerelease: false
   version_requirements: !ruby/object:Gem::Requirement
     requirements:
     - - "~>"
       - !ruby/object:Gem::Version
-        version: '10.0'
+        version: '13.0'
 - !ruby/object:Gem::Dependency
   name: rspec
   requirement: !ruby/object:Gem::Requirement
@@ -171,6 +171,7 @@ executables: []
 extensions: []
 extra_rdoc_files: []
 files:
+- ".circleci/config.yml"
 - ".github/CODEOWNERS"
 - ".gitignore"
 - ".ruby-version"
@@ -181,6 +182,7 @@ files:
 - LICENSE.txt
 - README.md
 - Rakefile
+- atlantis.yaml
 - danger-android_lint.gemspec
 - lib/android_lint/gem_version.rb
 - lib/android_lint/plugin.rb
@@ -188,15 +190,17 @@ files:
 - lib/danger_plugin.rb
 - spec/android_lint_spec.rb
 - spec/fixtures/lint-result-empty.xml
+- spec/fixtures/lint-result-for-pr.xml
 - spec/fixtures/lint-result-with-everything.xml
 - spec/fixtures/lint-result-with-special-chars.xml
 - spec/fixtures/lint-result-without-fatal.xml
+- spec/fixtures/pr-diff.diff
 - spec/spec_helper.rb
 homepage: https://github.com/loadsmart/danger-android_lint
 licenses:
 - MIT
 metadata: {}
-post_install_message: 
+post_install_message:
 rdoc_options: []
 require_paths:
 - lib
@@ -211,16 +215,17 @@ required_rubygems_version: !ruby/object:Gem::Requirement
     - !ruby/object:Gem::Version
       version: '0'
 requirements: []
-rubyforge_project: 
-rubygems_version: 2.5.2.1
-signing_key: 
+rubygems_version: 3.2.22
+signing_key:
 specification_version: 4
 summary: Lint files of a gradle based Android project. This is done using the Android's
   Lint tool. Results are passed out as tables in markdown.
 test_files:
 - spec/android_lint_spec.rb
 - spec/fixtures/lint-result-empty.xml
+- spec/fixtures/lint-result-for-pr.xml
 - spec/fixtures/lint-result-with-everything.xml
 - spec/fixtures/lint-result-with-special-chars.xml
 - spec/fixtures/lint-result-without-fatal.xml
+- spec/fixtures/pr-diff.diff
 - spec/spec_helper.rb