Closed cfergeau closed 2 weeks ago
New changes are detected. LGTM label has been removed.
I dropped the SetTrustedProxies
commit as it was alreadypresent in git master, and I added another commit to fix recent linting errors.
@nirs: changing LGTM is restricted to collaborators
[APPROVALNOTIFIER] This PR is APPROVED
This pull-request has been approved by: baude, nirs
The full list of commands accepted by this bot can be found here.
The pull request process is described here
I think it will be better to split the commit handling possible integer overflow to another PR, and merge the easy changes now.
I think it will be better to split the commit handling possible integer overflow to another PR, and merge the easy changes now.
Yep, I was also getting to that conclusion, I initially thought the overflow changes would be a quick fix ;)
I think it will be better to split the commit handling possible integer overflow to another PR, and merge the easy changes now.
Yep, I was also getting to that conclusion, I initially thought the overflow changes would be a quick fix ;)
This fixes minor (confusing) errors in the documentation and some lint errors.