Skip to content

Commit

Permalink
Merge pull request #1018 from lf-lang/fix-1016
Browse files Browse the repository at this point in the history
Suppress line-too-long warnings in the Python target.
  • Loading branch information
edwardalee authored Mar 9, 2022
2 parents ec5878d + f93b150 commit f2a1553
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -295,7 +295,7 @@ public Strategy getOutputReportingStrategy() {
*/
private boolean shouldIgnore(PylintMessage message) {
// Code generation does not preserve whitespace, so this check is unreliable.
if (message.symbol.equals("trailing-whitespace")) return true;
if (message.symbol.equals("trailing-whitespace") || message.symbol.equals("line-too-long")) return true;
// This filters out Pylint messages concerning missing members in types defined by protocol buffers.
// FIXME: Make this unnecessary, perhaps using https://github.com/nelfin/pylint-protobuf.
Matcher matcher = PylintNoNamePattern.matcher(message.message);
Expand Down

0 comments on commit f2a1553

Please sign in to comment.