danger: Failed to rescue in block in submit_inline_comments_for_kind!
Report
Running danger using gem versions
Using danger 6.0.0
Using danger-plugin-api 1.0.0
...
Using danger-android_lint 0.0.6
Using ox 2.10.0
Using danger-checkstyle_format 0.1.1
...
Using gitlab 4.10.0
Using danger-gitlab 7.0.0
results in
Server responded with code 400, message: 400 (Bad request) "Note {:line_code=>["can't be blank", "must be a valid line code"]}" not given. Request URI: https://git.domain.com/api/v4/projects/my_project/merge_requests/2/discussions
body: <table data-meta="generated_by_danger">
<tbody>
<tr>
<td>:warning:</td>
<td width="100%" data-sticky="false">app/src/main/java/com/domain/project/SomeClass.kt#L1Needless blank line(s)</td>
</tr>
</tbody>
</table>
position: {:position_type=>"text", :new_path=>"app/src/main/java/com/domain/project/SomeClass.kt", :new_line=>1, :base_sha=>"727a60e3b24bfcba507b3d8687b1a428c73f5e4d", :start_sha=>"727a60e3b24bfcba507b3d8687b1a428c73f5e4d", :head_sha=>"378f448b7a41cc43b67a98855fd2c39e1398cb22"}
/var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:425:in `rescue in block in submit_inline_comments_for_kind!': undefined method `[]' for nil:NilClass (NoMethodError)
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:402:in `block in submit_inline_comments_for_kind!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:374:in `reject'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:374:in `submit_inline_comments_for_kind!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:335:in `submit_inline_comments!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:187:in `update_pull_request_with_inline_comments!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:154:in `update_pull_request!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/dangerfile.rb:247:in `post_results'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/dangerfile.rb:278:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/executor.rb:29:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/commands/runner.rb:72:in `run'
from /var/lib/gems/2.6.0/gems/claide-1.0.2/lib/claide/command.rb:334:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/bin/danger:5:in `<top (required)>'
from /usr/local/bin/danger:23:in `load'
from /usr/local/bin/danger:23:in `<main>'
/var/lib/gems/2.6.0/gems/gitlab-4.10.0/lib/gitlab/request.rb:67:in `validate': Server responded with code 400, message: Bad Request. Request URI: https://git.domain.com/api/v4/projects/my_project/merge_requests/2/discussions/2a8634d25f816bbefefe629998db4533905a8a7f/notes/36227 (Gitlab::Error::BadRequest)
from /var/lib/gems/2.6.0/gems/gitlab-4.10.0/lib/gitlab/request.rb:46:in `block (2 levels) in <class:Request>'
from /var/lib/gems/2.6.0/gems/gitlab-4.10.0/lib/gitlab/client/merge_requests.rb:284:in `update_merge_request_discussion_note'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:422:in `block in submit_inline_comments_for_kind!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:374:in `reject'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:374:in `submit_inline_comments_for_kind!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:335:in `submit_inline_comments!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:187:in `update_pull_request_with_inline_comments!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/request_sources/gitlab.rb:154:in `update_pull_request!'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/dangerfile.rb:247:in `post_results'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/dangerfile.rb:278:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/danger_core/executor.rb:29:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/lib/danger/commands/runner.rb:72:in `run'
from /var/lib/gems/2.6.0/gems/claide-1.0.2/lib/claide/command.rb:334:in `run'
from /var/lib/gems/2.6.0/gems/danger-6.0.0/bin/danger:5:in `<top (required)>'
from /usr/local/bin/danger:23:in `load'
from /usr/local/bin/danger:23:in `<main>'
Your Environment
- Which CI are you running on?
GitLab 11.8.0 (ce13864)
- What is your Dangerfile?
declared_trivial = (gitlab.mr_title + gitlab.mr_body).include?("trivial")
warn("MR is classed as Work in Progress") if gitlab.mr_title.include? "[WIP]"
failure "Please provide a summary in the Merge Request description" if gitlab.mr_body.length < 5
# Warn when detekt config has changed
warn("Detekt config has changed") if git.modified_files.include? "detekt-config.yml"
# Warn when there is a big PR
warn("Big PR") if git.lines_of_code > 500
# detekt
checkstyle_format.base_path = Dir.pwd
checkstyle_format.report "/builds/hireme/hireme-android/app/build/reports/detekt/detekt.xml"
# AndroidLint
android_lint.report_file = "app/build/reports/lint-results.xml"
android_lint.lint(inline_mode: true)
About this issue
- Original URL
- State: open
- Created 5 years ago
- Comments: 16 (8 by maintainers)
Hopefully! Thanks again for the feedback!