Skip to content

Commit c53ad28

Browse files
committed
csfilter: abstract out precise numbers in Pylint output
Reported-By: Pavel Raiskup Fixes: https://github.com/kdudka/csdiff/issues/5
1 parent 5b15dac commit c53ad28

8 files changed

+1019
-0
lines changed

csfilter.cc

Lines changed: 3 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -113,6 +113,9 @@ struct MsgFilter::Private {
113113
// unify (per build random) names of temporary variables
114114
addMsgFilter("COMPILER_WARNING", "_tmp[0-9]+_", "_tmp_");
115115

116+
// pylint: "Too many lines in module (1152/1000)" etc.
117+
addMsgFilter("PROSPECTOR_WARNING", " \\([0-9]+/[0-9]+\\)$", "");
118+
116119
// "__coverity_strcmp" -> "strcmp", etc.
117120
addMsgFilter("", "__coverity_", "");
118121

tests/CMakeLists.txt

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -208,6 +208,7 @@ test_csdiff(diff-misc 06)
208208
test_csdiff(diff-misc 07)
209209
test_csdiff(diff-misc 08)
210210
test_csdiff(diff-misc 09-shellcheck-raw)
211+
test_csdiff(diff-misc 10-pylint-copr)
211212

212213
# cstrans-df-run tests
213214
tests_cstrans_df_run(0001)

tests/diff-misc/10-pylint-copr-add-z.err

Whitespace-only changes.

tests/diff-misc/10-pylint-copr-add.err

Whitespace-only changes.
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
Error: PROSPECTOR_WARNING:
2+
/coprs_frontend/commands/rawhide_to_release.py:8: W0611[pylint]: Unused builds_logic imported from coprs.logic
Lines changed: 2 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,2 @@
1+
Error: PROSPECTOR_WARNING:
2+
/coprs_frontend/commands/rawhide_to_release.py:8: W0611[pylint]: Unused builds_logic imported from coprs.logic

tests/diff-misc/10-pylint-copr-new.err

Lines changed: 505 additions & 0 deletions
Large diffs are not rendered by default.

tests/diff-misc/10-pylint-copr-old.err

Lines changed: 506 additions & 0 deletions
Large diffs are not rendered by default.

0 commit comments

Comments
 (0)