Skip to content

Conversation

schlamar
Copy link
Contributor

  • start collection from pytest_load_initial_conftests hook
  • dist master should start collecting, too
  • create test case using conftest
  • better solution for mocking up an option object (?)

@lahwran
Copy link

lahwran commented Apr 23, 2014

cool, I have no moral attachment to any particular solution, as long as it satisfies everyone. what I did is good enough for me for now, so don't rush this on my account.

@schlamar
Copy link
Contributor Author

@lahwran Can you please confirm that my solution is working, too?

@schlamar schlamar changed the title [WIP] use pytest_load_initial_conftest Use pytest_load_initial_conftest Apr 24, 2014
schlamar added a commit that referenced this pull request Apr 24, 2014
@schlamar schlamar merged commit 5bec42a into master Apr 24, 2014
@schlamar schlamar deleted the load-initial-conftest branch April 24, 2014 20:09
@sontek
Copy link

sontek commented Jun 4, 2014

@schlamar @lahwran I just tested and this is working for me. Any way we can get a new release of pytest-cov pushed to pypi?

@schlamar
Copy link
Contributor Author

schlamar commented Jun 5, 2014

@sontek I want to settle #9 before a release.

@sontek
Copy link

sontek commented Jun 5, 2014

@schlamar Cool, I keep an eye on it! When we added a conftest our coverage dropped and that made us sad :P

cclauss added a commit to cclauss/pytest-cov that referenced this pull request Feb 19, 2023
# This is the 1st commit message:

Add Python 3.11 and PyPy 3.9 to the testing
# The commit message pytest-dev#2 will be skipped:

# tox.ini: Add py311 and pypy39

# The commit message pytest-dev#3 will be skipped:

# pytest72

# The commit message pytest-dev#4 will be skipped:

# xdist302
#
# Co-authored-by: Mason Lin <[email protected]>

# The commit message pytest-dev#5 will be skipped:

# coverage65
#
# Co-authored-by: Mason Lin <[email protected]>

# The commit message pytest-dev#6 will be skipped:

# Drop py36 and upgrade to pytest-xdist==3.1.0
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