/matplotlib

Don't blacklist test_usetex using pytest #7012

Merged
merged 1 commit into from Aug 31, 2016

Conversation

Reviewers
No reviews
Assignees
No one assigned
Labels
Testing
Projects
None yet
Milestone
  2.1 (next point r...
3 participants
@jenshnielsen @Kojoley @QuLogic
@jenshnielsen
Owner

jenshnielsen commented Aug 31, 2016

The tests where fixed in #6918 but not enabled for pytest
@jenshnielsen jenshnielsen Don't blacklist test_usetex using pytest 
The tests where fixed in #6918 but not enabled for pytest

All checks have passed

3 successful checks
continuous-integration/appveyor/pr — AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr — The Travis CI build passed
Details
coverage/coveralls — Coverage remained the same at 70.617%
Details
8e4dd95

jenshnielsen added the Testing label Aug 31, 2016

jenshnielsen added this to the 2.1 (next point release) milestone Aug 31, 2016

jenshnielsen added the needs_review label Aug 31, 2016

@Kojoley
Member

Kojoley commented Aug 31, 2016

Maybe it is better to completely remove white and black lists support?
@jenshnielsen
Owner

jenshnielsen commented Aug 31, 2016

I would be ok with that. But that would remove the no pep 8 option too as far as I can see? I don't know if we want to keep that?
@Kojoley
Member

Kojoley commented Aug 31, 2016

No, pep8 will not be affected. Only --collect-filter will gone.
@Kojoley
Member

Kojoley commented Aug 31, 2016

Nevermind, postpone w/b lists removal until migration to pytest-pep8 plugin from test_coding_standards test.

@QuLogic QuLogic merged commit bb93047 into matplotlib:master Aug 31, 2016
3 checks passed

3 checks passed

continuous-integration/appveyor/pr AppVeyor build succeeded
Details
continuous-integration/travis-ci/pr The Travis CI build passed
Details
coverage/coveralls Coverage remained the same at 70.617%
Details

QuLogic removed the needs_review label Aug 31, 2016

jenshnielsen deleted the jenshnielsen:enable_usetex_pytest branch Aug 31, 2016

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment