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

Initial #1

Merged
merged 14 commits into from Jan 3, 2014
Merged

Initial #1

merged 14 commits into from Jan 3, 2014

Conversation

amakhnach
Copy link
Contributor

  • Made implementation of plugin.
  • Added --show-fixture-duplicates option.
  • Added tests to check that --show-fixture-duplicates works.



def pytest_cmdline_main(config):
"""Pytest commandline hook which called for performing the main command line action. The default
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

document what you do and why, don't describe pytest's internals

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 90bd869 on initial into * on master*.

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling 2efcd8f on initial into * on master*.



def _show_fixture_duplicates_main(config, session):
"""Prepearint fixture duplicates for output."""
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

typo

@coveralls
Copy link

Coverage Status

Changes Unknown when pulling b3ff6b4 on initial into * on master*.

bubenkoff added a commit that referenced this pull request Jan 3, 2014
@bubenkoff bubenkoff merged commit 329f4e8 into master Jan 3, 2014
@bubenkoff bubenkoff deleted the initial branch January 3, 2014 13:09
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.

None yet

3 participants