Skip to content

Commit

Permalink
Merge pull request #3576 from RonnyPfannschmidt/addmarker-api
Browse files Browse the repository at this point in the history
fix addmarker - extract mark from markdecorator
  • Loading branch information
nicoddemus authored Jun 13, 2018
2 parents e0a1da4 + 8d07220 commit 94c41be
Show file tree
Hide file tree
Showing 29 changed files with 63 additions and 1,077 deletions.
35 changes: 15 additions & 20 deletions CONTRIBUTING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -162,11 +162,11 @@ Preparing Pull Requests
Short version
~~~~~~~~~~~~~

#. Fork the repository;
#. enable and install pre-commit https://pre-commit.com/ to ensure styleguides and codechecks are followed
#. Target ``master`` for bugfixes and doc changes;
#. Fork the repository.
#. Enable and install `pre-commit <https://pre-commit.com>`_ to ensure style-guides and code checks are followed.
#. Target ``master`` for bugfixes and doc changes.
#. Target ``features`` for new features or functionality changes.
#. Follow **PEP-8**. There's a ``tox`` command to help fixing it: ``tox -e fix-lint``.
#. Follow **PEP-8** for naming and `black <https://github.com/ambv/black>`_ for formatting.
#. Tests are run using ``tox``::

tox -e linting,py27,py36
Expand All @@ -177,7 +177,7 @@ Short version
and one of ``bugfix``, ``removal``, ``feature``, ``vendor``, ``doc`` or
``trivial`` for the issue type.
#. Unless your change is a trivial or a documentation fix (e.g., a typo or reword of a small section) please
add yourself to the ``AUTHORS`` file, in alphabetical order;
add yourself to the ``AUTHORS`` file, in alphabetical order.


Long version
Expand Down Expand Up @@ -217,15 +217,15 @@ Here is a simple overview, with pytest-specific bits:
If you need some help with Git, follow this quick start
guide: https://git.wiki.kernel.org/index.php/QuickStart

#. install pre-commit and install its hook on the pytest repo
#. Install `pre-commit <https://pre-commit.com>`_ and its hook on the pytest repo::

https://pre-commit.com/ is a framework for managing and maintaining multi-language pre-commit hooks
pytest uses pre-commit to ensure code-style and code formatting is the same
$ pip install --user pre-commit
$ pre-commit install

$ pip install --user pre-commit
$ pre-commit install
Afterwards ``pre-commit`` will run whenever you commit.

Afterwards pre-commit will run whenever you commit.
https://pre-commit.com/ is a framework for managing and maintaining multi-language pre-commit hooks
to ensure code-style and code formatting is consistent.

#. Install tox

Expand All @@ -245,15 +245,7 @@ Here is a simple overview, with pytest-specific bits:
This command will run tests via the "tox" tool against Python 2.7 and 3.6
and also perform "lint" coding-style checks.

#. You can now edit your local working copy. Please follow PEP-8.

You can now make the changes you want and run the tests again as necessary.

If you have too much linting errors, try running::

$ tox -e fix-lint

To fix pep8 related errors.
#. You can now edit your local working copy and run the tests again as necessary. Please follow PEP-8 for naming.

You can pass different options to ``tox``. For example, to run tests on Python 2.7 and pass options to pytest
(e.g. enter pdb on failure) to pytest you can do::
Expand All @@ -264,6 +256,9 @@ Here is a simple overview, with pytest-specific bits:

$ tox -e py36 -- testing/test_config.py


When committing, ``pre-commit`` will re-format the files if necessary.

#. Commit and push once your tests pass and you are happy with your change(s)::

$ git commit -a -m "<commit message>"
Expand Down
1 change: 1 addition & 0 deletions changelog/3555.bugfix.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
Fix regression in ``Node.add_marker`` by extracting the mark object of a ``MarkDecorator``.
1 change: 1 addition & 0 deletions changelog/3576.feature.rst
Original file line number Diff line number Diff line change
@@ -0,0 +1 @@
``Node.add_marker`` now supports an append=True/False to determine whether the mark comes last (default) or first.
2 changes: 1 addition & 1 deletion doc/en/assert.rst
Original file line number Diff line number Diff line change
Expand Up @@ -91,7 +91,7 @@ In the context manager form you may use the keyword argument
``message`` to specify a custom failure message::

>>> with raises(ZeroDivisionError, message="Expecting ZeroDivisionError"):
... pass
... pass
... Failed: Expecting ZeroDivisionError

If you want to write test code that works on Python 2.4 as well,
Expand Down
17 changes: 0 additions & 17 deletions doc/en/check_sphinx.py

This file was deleted.

44 changes: 0 additions & 44 deletions doc/en/genapi.py

This file was deleted.

117 changes: 0 additions & 117 deletions doc/en/test/attic.rst

This file was deleted.

17 changes: 0 additions & 17 deletions doc/en/test/config.html

This file was deleted.

17 changes: 0 additions & 17 deletions doc/en/test/dist.html

This file was deleted.

17 changes: 0 additions & 17 deletions doc/en/test/extend.html

This file was deleted.

33 changes: 0 additions & 33 deletions doc/en/test/index.rst

This file was deleted.

13 changes: 0 additions & 13 deletions doc/en/test/mission.rst

This file was deleted.

Loading

0 comments on commit 94c41be

Please sign in to comment.