Skip to content

Commit

Permalink
Update doc and sip coverage test thresholds
Browse files Browse the repository at this point in the history
  • Loading branch information
nyalldawson committed Dec 10, 2015
1 parent b23ddfe commit c854d05
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 3 deletions.
2 changes: 1 addition & 1 deletion tests/src/python/test_qgsdoccoverage.py
Expand Up @@ -27,7 +27,7 @@
# DON'T RAISE THIS THRESHOLD!!!
# (changes which lower this threshold are welcomed though!)

ACCEPTABLE_MISSING_DOCS = 3907
ACCEPTABLE_MISSING_DOCS = 3873


class TestQgsDocCoverage(TestCase):
Expand Down
4 changes: 2 additions & 2 deletions tests/src/python/test_qgssipcoverage.py
Expand Up @@ -37,8 +37,8 @@
# DON'T RAISE THIS THRESHOLD!!!
# (changes which lower this threshold are welcomed though!)

ACCEPTABLE_MISSING_CLASSES = 85
ACCEPTABLE_MISSING_MEMBERS = 267
ACCEPTABLE_MISSING_CLASSES = 80
ACCEPTABLE_MISSING_MEMBERS = 266


class TestQgsSipCoverage(TestCase):
Expand Down

2 comments on commit c854d05

@SebDieBln
Copy link
Contributor

Choose a reason for hiding this comment

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

I think it is just bad luck to modify this poorly indented file.

@nyalldawson
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

@jef-n do you know what's happening with the indentation test here? I've run this commit through prepare_commit with no changes. astyle-all doesn't make any difference either.

Could it be that the version of autopep8 I have installed is too old/skipping the space-after-# formatting?

Please sign in to comment.