Skip to content

Merge master into features #6563

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

Merged
merged 41 commits into from
Jan 24, 2020
Merged
Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
41 commits
Select commit Hold shift + click to select a range
62db3f7
typing: fix/adjust _code.source.getfslineno
blueyed Jan 21, 2020
bd6ba3f
typing: Session.__init__
blueyed Jan 21, 2020
ad02f6f
Merge pull request #6525 from blueyed/typing-session
blueyed Jan 22, 2020
8fa57c8
tests: improve test for `nose.raises`
blueyed Jan 21, 2020
ef112fd
Revert "Revert "Fix type errors after adding types to the `py` depend…
blueyed Jan 21, 2020
1350c60
Node.location: handle str with _node_location_to_relpath
blueyed Jan 21, 2020
9c7b3c5
typing: PyobjMixin.reportinfo, getfslineno
blueyed Jan 21, 2020
252eae5
tests: fix/harden test_record_property
blueyed Jan 23, 2020
eb7a57f
Merge pull request #6542 from blueyed/fix-test_record_property
blueyed Jan 23, 2020
e7444bb
tests: remove unnecessary `-rw` option
blueyed Jan 23, 2020
10e243d
Merge pull request #6543 from blueyed/rm-rw
blueyed Jan 23, 2020
b63cb18
doc: remove costlysetup example
blueyed Jan 23, 2020
0b6258a
PyCollector.collect: use explicit cast to `str`
bluetech Jan 23, 2020
8521503
Merge pull request #6527 from blueyed/typing-getfslineno
blueyed Jan 23, 2020
863bab5
Merge pull request #6544 from blueyed/doc-rm-costlysetup
blueyed Jan 23, 2020
9dcdea5
Rewrite Item.location to be clearer with regard to types
bluetech Jan 23, 2020
6b13379
Merge pull request #6521 from blueyed/harden-nose-raises
blueyed Jan 23, 2020
03bc8ab
config: typing for create_terminal_writer, re-export TerminalWriter
blueyed Jan 23, 2020
a8d67f5
ci: codecov: add flags for GHA/Travis
blueyed Jan 23, 2020
8ca8d25
Merge pull request #6545 from blueyed/terminalwriter
blueyed Jan 23, 2020
55ebd9f
doc: minor fixes for the release process
blueyed Jan 22, 2020
38538c6
Merge pull request #6548 from blueyed/ci-codecov-flag-gha
blueyed Jan 23, 2020
83451b5
doc/en/fixture.rst: chdir back to previous directory
blueyed Jan 23, 2020
cdd6f86
bug-fix fixes
blueyed Jan 23, 2020
c051a9e
ci: GHA: separate jobs for linting, docs, doctesting
blueyed Jan 23, 2020
79ae86c
tests: fix test_repr_traceback_with_invalid_cwd
blueyed Jan 21, 2020
79d00ab
Merge pull request #6549 from blueyed/fix-release-minor
blueyed Jan 24, 2020
0d4f479
Merge pull request #6552 from blueyed/ci-linting
blueyed Jan 24, 2020
f8654e6
ci: Travis: removing linting,docs,doctesting
blueyed Jan 24, 2020
e5a362d
ci: GHA: use scripts/report-coverage.sh
blueyed Jan 24, 2020
5dcd24f
Merge pull request #6553 from blueyed/ci-gha-use-report-coverage.sh
blueyed Jan 24, 2020
934f389
ci: GHA: codecov: set CODECOV_NAME, OS in flags
blueyed Jan 24, 2020
fe895a4
Merge pull request #6559 from blueyed/ci-travis-remove-linting
blueyed Jan 24, 2020
e2fa78c
ci: remove Azure config/scripts
blueyed Jan 24, 2020
5865520
Merge pull request #6561 from blueyed/ci-gha-name-flag
blueyed Jan 24, 2020
192d3ad
tests: add test_fixture_arg_ordering
blueyed Jan 24, 2020
8a8f9bd
Merge pull request #6562 from blueyed/ci-rm-azure
blueyed Jan 24, 2020
2c32dad
Merge pull request #6554 from blueyed/test_fixture_arg_ordering
blueyed Jan 24, 2020
040a61e
Merge pull request #6550 from blueyed/doc-cleandir-cd-back
blueyed Jan 24, 2020
09ab5fd
Merge pull request #6529 from blueyed/fix-test_repr_traceback_with_in…
blueyed Jan 24, 2020
09bdbff
Merge master into features
blueyed Jan 24, 2020
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
32 changes: 17 additions & 15 deletions .github/workflows/main.yml
Original file line number Diff line number Diff line change
Expand Up @@ -45,6 +45,8 @@ jobs:
"macos-py38",

"linting",
"docs",
"doctesting",
]

include:
Expand Down Expand Up @@ -114,7 +116,17 @@ jobs:
- name: "linting"
python: "3.7"
os: ubuntu-latest
tox_env: "linting,docs,doctesting"
tox_env: "linting"
skip_coverage: true
- name: "docs"
python: "3.7"
os: ubuntu-latest
tox_env: "docs"
skip_coverage: true
- name: "doctesting"
python: "3.7"
os: ubuntu-latest
tox_env: "doctesting"

steps:
- uses: actions/checkout@v1
Expand Down Expand Up @@ -144,21 +156,11 @@ jobs:
run: |
python scripts/append_codecov_token.py

- name: Combine coverage
- name: Report coverage
if: (!matrix.skip_coverage)
run: |
python -m coverage combine
python -m coverage xml

- name: Codecov upload
if: (!matrix.skip_coverage)
uses: codecov/codecov-action@v1
with:
token: ${{ secrets.codecov }}
file: ./coverage.xml
flags: ${{ runner.os }}
fail_ci_if_error: false
name: ${{ matrix.name }}
env:
CODECOV_NAME: ${{ matrix.name }}
run: bash scripts/report-coverage.sh -F GHA,${{ runner.os }}

deploy:
if: github.event_name == 'push' && startsWith(github.event.ref, 'refs/tags') && github.repository == 'pytest-dev/pytest'
Expand Down
2 changes: 1 addition & 1 deletion .pre-commit-config.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -64,7 +64,7 @@ repos:
_code\.|
builtin\.|
code\.|
io\.(BytesIO|saferepr)|
io\.(BytesIO|saferepr|TerminalWriter)|
path\.local\.sysfind|
process\.|
std\.
Expand Down
7 changes: 1 addition & 6 deletions .travis.yml
Original file line number Diff line number Diff line change
Expand Up @@ -47,11 +47,6 @@ jobs:
python: '3.5.1'
dist: trusty

- env: TOXENV=linting,docs,doctesting PYTEST_COVERAGE=1
cache:
directories:
- $HOME/.cache/pre-commit

before_script:
- |
# Do not (re-)upload coverage with cron runs.
Expand All @@ -71,7 +66,7 @@ script: tox
after_success:
- |
if [[ "$PYTEST_COVERAGE" = 1 ]]; then
env CODECOV_NAME="$TOXENV-$TRAVIS_OS_NAME" scripts/report-coverage.sh
env CODECOV_NAME="$TOXENV-$TRAVIS_OS_NAME" scripts/report-coverage.sh -F Travis
fi

notifications:
Expand Down
6 changes: 3 additions & 3 deletions CONTRIBUTING.rst
Original file line number Diff line number Diff line change
Expand Up @@ -166,7 +166,7 @@ Short version

#. 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 ``master`` for bug fixes and doc changes.
#. Target ``features`` for new features or functionality changes.
#. Follow **PEP-8** for naming and `black <https://github.com/psf/black>`_ for formatting.
#. Tests are run using ``tox``::
Expand Down Expand Up @@ -212,7 +212,7 @@ Here is a simple overview, with pytest-specific bits:

$ git checkout -b your-feature-branch-name features

Given we have "major.minor.micro" version numbers, bugfixes will usually
Given we have "major.minor.micro" version numbers, bug fixes will usually
be released in micro releases whereas features will be released in
minor releases and incompatible changes in major releases.

Expand Down Expand Up @@ -294,7 +294,7 @@ Here is a simple overview, with pytest-specific bits:
compare: your-branch-name

base-fork: pytest-dev/pytest
base: master # if it's a bugfix
base: master # if it's a bug fix
base: features # if it's a feature


Expand Down
4 changes: 2 additions & 2 deletions HOWTORELEASE.rst
Original file line number Diff line number Diff line change
@@ -1,14 +1,14 @@
Release Procedure
-----------------

Our current policy for releasing is to aim for a bugfix every few weeks and a minor release every 2-3 months. The idea
Our current policy for releasing is to aim for a bug-fix release every few weeks and a minor release every 2-3 months. The idea
is to get fixes and new features out instead of trying to cram a ton of features into a release and by consequence
taking a lot of time to make a new one.

.. important::

pytest releases must be prepared on **Linux** because the docs and examples expect
to be executed in that platform.
to be executed on that platform.

#. Create a branch ``release-X.Y.Z`` with the version for the release.

Expand Down
80 changes: 0 additions & 80 deletions azure-pipelines.yml

This file was deleted.

2 changes: 1 addition & 1 deletion changelog/README.rst
Original file line number Diff line number Diff line change
Expand Up @@ -15,7 +15,7 @@ Each file should be named like ``<ISSUE>.<TYPE>.rst``, where

* ``feature``: new user facing features, like new command-line options and new behavior.
* ``improvement``: improvement of existing functionality, usually without requiring user intervention (for example, new fields being written in ``--junitxml``, improved colors in terminal, etc).
* ``bugfix``: fixes a reported bug.
* ``bugfix``: fixes a bug.
* ``doc``: documentation improvement, like rewording an entire session or adding missing docs.
* ``deprecation``: feature deprecation.
* ``breaking``: a change which may break existing suites, such as feature removal or behavior change.
Expand Down
4 changes: 2 additions & 2 deletions doc/en/changelog.rst
Original file line number Diff line number Diff line change
Expand Up @@ -2357,7 +2357,7 @@ Deprecations and Removals
- `#4036 <https://github.com/pytest-dev/pytest/issues/4036>`_: The ``item`` parameter of ``pytest_warning_captured`` hook is now documented as deprecated. We realized only after
the ``3.8`` release that this parameter is incompatible with ``pytest-xdist``.

Our policy is to not deprecate features during bugfix releases, but in this case we believe it makes sense as we are
Our policy is to not deprecate features during bug-fix releases, but in this case we believe it makes sense as we are
only documenting it as deprecated, without issuing warnings which might potentially break test suites. This will get
the word out that hook implementers should not use this parameter at all.

Expand Down Expand Up @@ -5380,7 +5380,7 @@ time or change existing behaviors in order to make them less surprising/more use
Thanks Daniel Grunwald for the report and Bruno Oliveira for the PR.

- (experimental) adapt more SEMVER style versioning and change meaning of
master branch in git repo: "master" branch now keeps the bugfixes, changes
master branch in git repo: "master" branch now keeps the bug fixes, changes
aimed for micro releases. "features" branch will only be released
with minor or major pytest releases.

Expand Down
2 changes: 1 addition & 1 deletion doc/en/development_guide.rst
Original file line number Diff line number Diff line change
Expand Up @@ -19,7 +19,7 @@ Branches

We have two long term branches:

* ``master``: contains the code for the next bugfix release.
* ``master``: contains the code for the next bug-fix release.
* ``features``: contains the code with new features for the next minor release.

The official repository usually does not contain topic branches, developers and contributors should create topic
Expand Down
20 changes: 0 additions & 20 deletions doc/en/example/costlysetup/conftest.py

This file was deleted.

1 change: 0 additions & 1 deletion doc/en/example/costlysetup/sub_a/__init__.py

This file was deleted.

2 changes: 0 additions & 2 deletions doc/en/example/costlysetup/sub_a/test_quick.py

This file was deleted.

1 change: 0 additions & 1 deletion doc/en/example/costlysetup/sub_b/__init__.py

This file was deleted.

6 changes: 0 additions & 6 deletions doc/en/example/costlysetup/sub_b/test_two.py

This file was deleted.

4 changes: 3 additions & 1 deletion doc/en/fixture.rst
Original file line number Diff line number Diff line change
Expand Up @@ -1042,11 +1042,13 @@ file:
import pytest


@pytest.fixture()
@pytest.fixture
def cleandir():
old_cwd = os.getcwd()
newpath = tempfile.mkdtemp()
os.chdir(newpath)
yield
os.chdir(old_cwd)
shutil.rmtree(newpath)

and declare its use in a test module via a ``usefixtures`` marker:
Expand Down
6 changes: 3 additions & 3 deletions scripts/release.minor.rst
Original file line number Diff line number Diff line change
Expand Up @@ -6,7 +6,7 @@ The pytest team is proud to announce the {version} release!
pytest is a mature Python testing tool with more than a 2000 tests
against itself, passing on many different interpreters and platforms.

This release contains a number of bugs fixes and improvements, so users are encouraged
This release contains a number of bug fixes and improvements, so users are encouraged
to take a look at the CHANGELOG:

https://docs.pytest.org/en/latest/changelog.html
Expand All @@ -15,7 +15,7 @@ For complete documentation, please visit:

https://docs.pytest.org/en/latest/

As usual, you can upgrade from pypi via:
As usual, you can upgrade from PyPI via:

pip install -U pytest

Expand All @@ -24,4 +24,4 @@ Thanks to all who contributed to this release, among them:
{contributors}

Happy testing,
The Pytest Development Team
The pytest Development Team
2 changes: 1 addition & 1 deletion scripts/report-coverage.sh
Original file line number Diff line number Diff line change
Expand Up @@ -15,4 +15,4 @@ python -m coverage xml
python -m coverage report -m
# Set --connect-timeout to work around https://github.com/curl/curl/issues/4461
curl -S -L --connect-timeout 5 --retry 6 -s https://codecov.io/bash -o codecov-upload.sh
bash codecov-upload.sh -Z -X fix -f coverage.xml
bash codecov-upload.sh -Z -X fix -f coverage.xml "$@"
21 changes: 0 additions & 21 deletions scripts/retry.cmd

This file was deleted.

Loading