Skip to content

TST: Fix test header in tox, stricter PEP 8#123

Merged
pllim merged 1 commit intoastropy:masterfrom
pllim:tst-header
Feb 18, 2021
Merged

TST: Fix test header in tox, stricter PEP 8#123
pllim merged 1 commit intoastropy:masterfrom
pllim:tst-header

Conversation

@pllim
Copy link
Copy Markdown
Member

@pllim pllim commented Feb 18, 2021

Test header to show properly in tox. Stop excluding files for PEP 8.

@pllim pllim added the enhancement New feature or request label Feb 18, 2021
@pllim pllim added this to the 0.3 milestone Feb 18, 2021
@pllim
Copy link
Copy Markdown
Member Author

pllim commented Feb 18, 2021

This should be uncontroversial enough, so merging without a review.

For future reference: tox sees only the root level conftest.py, while we need astrowidgets/conftest.py for the Astropy test runner, so that's why there are two copies. If we decide we don't need to run astrowidgets.tests(), we can remove the test runner from _astropy_init.py and also remove astrowidgets/conftest.py.

@pllim pllim merged commit fc9a27d into astropy:master Feb 18, 2021
@pllim pllim deleted the tst-header branch February 18, 2021 20:18
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant