Skip to content

don't allow pylint to fail any more#1994

Merged
richardjgowers merged 1 commit intodevelopfrom
pylint2
Jul 17, 2018
Merged

don't allow pylint to fail any more#1994
richardjgowers merged 1 commit intodevelopfrom
pylint2

Conversation

@richardjgowers
Copy link
Copy Markdown
Member

Fixes #

Changes made in this Pull Request:

PR Checklist

  • Tests?
  • Docs?
  • CHANGELOG updated?
  • Issue raised/referenced?

@orbeckst
Copy link
Copy Markdown
Member

I'll wait for Travis... feel free to ping me once it's done.

Copy link
Copy Markdown
Member

@orbeckst orbeckst left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lint passes on Travis... assuming everything else passes (why shouldn't it... but wait), I am ok with this (as an amendment to PR #1993 )

@richardjgowers richardjgowers merged commit d7cfc27 into develop Jul 17, 2018
@richardjgowers richardjgowers deleted the pylint2 branch July 17, 2018 22:31
@codecov
Copy link
Copy Markdown

codecov bot commented Jul 17, 2018

Codecov Report

Merging #1994 into develop will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff            @@
##           develop    #1994   +/-   ##
========================================
  Coverage    88.48%   88.48%           
========================================
  Files          142      142           
  Lines        17202    17202           
  Branches      2635     2635           
========================================
  Hits         15221    15221           
  Misses        1385     1385           
  Partials       596      596

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 5f151b4...41bed97. Read the comment docs.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants