Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: pytest-dev/pluggy
Failed to load repositories. Confirm that selected base ref is valid, then try again.
Loading
base: 0.8.0
Choose a base ref
...
head repository: pytest-dev/pluggy
Failed to load repositories. Confirm that selected head ref is valid, then try again.
Loading
compare: 0.8.1
Choose a head ref
  • 8 commits
  • 5 files changed
  • 4 contributors

Commits on Sep 30, 2018

  1. Copy the full SHA
    f0d6f87 View commit details

Commits on Oct 16, 2018

  1. Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
    Copy the full SHA
    bee289c View commit details
  2. Merge pull request #180 from nicoddemus/release-0.8.0

    Preparing release 0.8.0
    nicoddemus authored Oct 16, 2018

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
    Copy the full SHA
    22d4417 View commit details
  3. Use conda-forge badge instead of anaconda

    The pluggy package is hosted on conda-forge, so it is nice to promote
    them and it also updates faster when there's a new release
    nicoddemus committed Oct 16, 2018

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
    Copy the full SHA
    3e21bfe View commit details
  4. Merge pull request #182 from nicoddemus/update-conda-badge

    Use conda-forge badge instead of anaconda
    nicoddemus authored Oct 16, 2018
    Copy the full SHA
    2724a58 View commit details
  5. Merge pull request #179 from nicoddemus/skip-deploy-without-tags

    Only generate 'deploy' jobs on Travis when a tag is present
    nicoddemus authored Oct 16, 2018

    Verified

    This commit was created on GitHub.com and signed with GitHub’s verified signature. The key has expired.
    Copy the full SHA
    72ee32f View commit details

Commits on Nov 9, 2018

  1. Merge pull request #181 from crazymerlyn/change_implprefix_warning_st…

    …acklevel
    
    Adjust stacklevel of implprefix warning
    goodboy authored Nov 9, 2018
    Copy the full SHA
    b40bcd8 View commit details
  2. Preparing release 0.8.1

    nicoddemus committed Nov 9, 2018
    Copy the full SHA
    317e992 View commit details
Showing with 24 additions and 3 deletions.
  1. +5 −0 .travis.yml
  2. +9 −0 CHANGELOG.rst
  3. +3 −3 README.rst
  4. +1 −0 pluggy/manager.py
  5. +6 −0 testing/test_pluginmanager.py
5 changes: 5 additions & 0 deletions .travis.yml
Original file line number Diff line number Diff line change
@@ -1,6 +1,11 @@
sudo: false
language: python

stages:
- test
- name: deploy
if: repo = pytest-dev/pluggy AND tag IS present

jobs:
include:
- python: '3.6'
9 changes: 9 additions & 0 deletions CHANGELOG.rst
Original file line number Diff line number Diff line change
@@ -1,3 +1,12 @@
pluggy 0.8.1 (2018-11-09)
=========================

Trivial/Internal Changes
------------------------

- `#166 <https://github.com/pytest-dev/pluggy/issues/166>`_: Add ``stacklevel=2`` to implprefix warning so that the reported location of warning is the caller of PluginManager.


pluggy 0.8.0 (2018-10-15)
=========================

6 changes: 3 additions & 3 deletions README.rst
Original file line number Diff line number Diff line change
@@ -2,7 +2,7 @@
pluggy - A minimalist production ready plugin system
====================================================

|pypi| |anaconda| |versions| |travis| |appveyor| |gitter| |black|
|pypi| |conda-forge| |versions| |travis| |appveyor| |gitter| |black|

This is the core framework used by the `pytest`_, `tox`_, and `devpi`_ projects.

@@ -75,8 +75,8 @@ A definitive example
.. |appveyor| image:: https://img.shields.io/appveyor/ci/pytestbot/pluggy/master.svg
:target: https://ci.appveyor.com/project/pytestbot/pluggy

.. |anaconda| image:: https://anaconda.org/conda-forge/pluggy/badges/version.svg
:target: https://anaconda.org/conda-forge/pluggy
.. |conda-forge| image:: https://img.shields.io/conda/vn/conda-forge/pluggy.svg
:target: https://anaconda.org/conda-forge/pytest

.. |gitter| image:: https://badges.gitter.im/pytest-dev/pluggy.svg
:alt: Join the chat at https://gitter.im/pytest-dev/pluggy
1 change: 1 addition & 0 deletions pluggy/manager.py
Original file line number Diff line number Diff line change
@@ -53,6 +53,7 @@ def __init__(self, project_name, implprefix=None):
"Support for the `implprefix` arg is now deprecated and will "
"be removed in an upcoming release. Please use HookimplMarker.",
DeprecationWarning,
stacklevel=2,
)
self._implprefix = implprefix
self._inner_hookexec = lambda hook, methods, kwargs: hook.multicall(
6 changes: 6 additions & 0 deletions testing/test_pluginmanager.py
Original file line number Diff line number Diff line change
@@ -579,6 +579,12 @@ def he_method1(self):
undo()


def test_implprefix_warning(recwarn):
PluginManager(hookspec.project_name, "hello_")
w = recwarn.pop(DeprecationWarning)
assert "test_pluginmanager.py" in w.filename


@pytest.mark.parametrize("include_hookspec", [True, False])
def test_prefix_hookimpl(include_hookspec):
with pytest.deprecated_call():