Negative value in mozlint tasks crashes the bot

[La0]

See https://firefoxci.taskcluster-artifacts.net/Zs4uj7w6Q3ylEuC4fPhhgA/0/public/logs/live_backing.log

[marco-c]

So I guess #175 was not complete?

[marco-c]

It's strange because we are supposed to convert -1 to None at

code-review/bot/code_review_bot/__init__.py

Line 73 in 210d010

self.line = positive_int("line", line)

, and then contains should return early when line is None:

code-review/bot/code_review_bot/revisions.py

Line 222 in 210d010

return True

.

Instead, it seems like we are getting to

code-review/bot/code_review_bot/revisions.py

Line 225 in 210d010

lines = set(range(issue.line, issue.line + issue.nb_lines))

.

[La0]

We are just not running the recent code in prod.
Doing a release now seems like a really good idea...

[marco-c]

Aaaah, that explains everything :P

[La0]

We should be good now that 1.0.7 has landed

[marco-c]

So we can close as duplicate of #168.

[La0]

I tested locally with current master, seems to work fine