X Tutup
Skip to content

Don't blacklist test_usetex using pytest#7012

Merged
QuLogic merged 1 commit intomatplotlib:masterfrom
jenshnielsen:enable_usetex_pytest
Aug 31, 2016
Merged

Don't blacklist test_usetex using pytest#7012
QuLogic merged 1 commit intomatplotlib:masterfrom
jenshnielsen:enable_usetex_pytest

Conversation

@jenshnielsen
Copy link
Copy Markdown
Member

The tests where fixed in #6918 but not enabled for pytest

The tests where fixed in matplotlib#6918 but not enabled for pytest
@jenshnielsen jenshnielsen added this to the 2.1 (next point release) milestone Aug 31, 2016
@Kojoley
Copy link
Copy Markdown
Member

Kojoley commented Aug 31, 2016

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

@jenshnielsen
Copy link
Copy Markdown
Member Author

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
Copy link
Copy Markdown
Member

Kojoley commented Aug 31, 2016

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

@Kojoley
Copy link
Copy Markdown
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
@jenshnielsen jenshnielsen deleted the enable_usetex_pytest branch August 31, 2016 19:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants

X Tutup