What version of Review Board are you running?
3.0.3
What version of Review Bot are you running?
1.0
What tool(s) is Review Bot running?
cpplint
What steps will reproduce the problem?
- Create a cpp/h file without a copyright header
- Upload to RB
- See that cpplint do not mark an issue for the missing copyright
What is the expected output? What do you see instead?
Add a "general comment issue" for line 0 issues.
Please provide any additional information below.
cpplint prints out line 0 for missing copyright. The reviewbot will ignore that as no one can change line 0.
cpplint output:
main.cpp:0: No copyright message found. You should have a line: "Copyright [year] <Copyright Owner>" [legal/copyright] [5]
It should be possible to add a general issue that will be displayed even if "comment on unmodified" is enabled. Example: clang-tidy tries to analyse the file but cannot finish the check because of compilation error. It should be reported as a general issue because the error could happen on unmodified line (as a result of another change). So it should show the compilation error for this.
Ping
Review for fix here: https://reviews.reviewboard.org/r/10093/
