Skip to content

Release 3.0.2#1896

Merged
nicoddemus merged 5 commits intopytest-dev:masterfrom
nicoddemus:release-3.0.2
Sep 2, 2016
Merged

Release 3.0.2#1896
nicoddemus merged 5 commits intopytest-dev:masterfrom
nicoddemus:release-3.0.2

Conversation

@nicoddemus
Copy link
Copy Markdown
Member

@nicoddemus nicoddemus commented Sep 1, 2016

Builds:

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 93.078% when pulling d512e7f on nicoddemus:release-3.0.2 into a094fb3 on pytest-dev:master.

* matthiasha

Happy testing,
The pytest Development Team
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

Comment thread _pytest/doctest.py
self.fixture_request = _setup_fixtures(self)
globs = dict(getfixture=self.fixture_request.getfixturevalue)
for name, value in self.fixture_request.getfuncargvalue('doctest_namespace').items():
for name, value in self.fixture_request.getfixturevalue('doctest_namespace').items():
Copy link
Copy Markdown
Member Author

Choose a reason for hiding this comment

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

I added the fix for #1898 in this PR because it was trivial and I didn't want to open a separate PR just for that, hope that's OK.

@nicoddemus
Copy link
Copy Markdown
Member Author

nicoddemus commented Sep 1, 2016

Pushed a new package:

EDIT: fixed AppVeyor link

@coveralls
Copy link
Copy Markdown

Coverage Status

Coverage remained the same at 93.078% when pulling 4e58c9a on nicoddemus:release-3.0.2 into a094fb3 on pytest-dev:master.

@nicoddemus
Copy link
Copy Markdown
Member Author

Anything else guys? I was thinking of pushing the release to PyPI later tonight, unless somebody wants more time to review this more carefully.

@The-Compiler
Copy link
Copy Markdown
Member

LGTM!

@nicoddemus nicoddemus merged commit 919f50a into pytest-dev:master Sep 2, 2016
@nicoddemus nicoddemus deleted the release-3.0.2 branch September 2, 2016 00:19
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.

3 participants