-
Notifications
You must be signed in to change notification settings - Fork 1.3k
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
ExternalBearWrap: Change JSON spec #2407
Conversation
a2d2a6d
to
f05de48
Compare
f05de48
to
1778263
Compare
This is a test run to investigate the failures on the CI |
28142b4
to
46cb551
Compare
origin=self, | ||
for result in output['results']: | ||
affected_code = (SourceRange.from_values( | ||
code_range['file'], |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
indentation off by a space here
d0d451a
to
5217af9
Compare
code_range['start'].get('column'), | ||
code_range.get('end', {}).get('line'), | ||
code_range.get('end', {}).get('column')) | ||
for code_range in result['affected_code']) |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
this line should be indented 6 spaces further, the ones above 10 spaces further
ack 5217af9 - so much better :) |
5217af9
to
cdb1553
Compare
cdb1553
to
6201b31
Compare
6201b31
to
f9dfb4c
Compare
ack 535bf93 |
ack f9dfb4c |
@rultor merge |
Fixes #2406
Fixes #2368