Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

fix addmarker - extract mark from markdecorator #3576

Merged
merged 17 commits into from
Jun 13, 2018
Merged
Show file tree
Hide file tree
Changes from 15 commits
Commits
File filter

Filter by extension

Filter by extension

Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
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