search

aragilar / pytest-mpi

Pytest plugin for working with MPI
https://pytest-mpi.readthedocs.io/
BSD 3-Clause "New" or "Revised" License
19 stars 7 forks source link

Update versioneer to 0.21 #36

Closed opoplawski closed 2 years ago

opoplawski commented 2 years ago

Should fix build with python 3.11, see https://bugzilla.redhat.com/show_bug.cgi?id=2028105

pep8speaks commented 2 years ago

Hello @opoplawski! Thanks for opening this PR. We checked the lines you've touched for PEP 8 issues, and found:

Line 342:80: E501 line too long (84 > 79 characters) Line 432:80: E501 line too long (80 > 79 characters) Line 435:80: E501 line too long (82 > 79 characters)

Line 194:80: E501 line too long (87 > 79 characters) Line 196:80: E501 line too long (86 > 79 characters) Line 224:80: E501 line too long (85 > 79 characters) Line 255:80: E501 line too long (87 > 79 characters) Line 257:80: E501 line too long (83 > 79 characters) Line 259:80: E501 line too long (86 > 79 characters) Line 664:80: E501 line too long (81 > 79 characters) Line 758:80: E501 line too long (85 > 79 characters) Line 848:80: E501 line too long (80 > 79 characters) Line 851:80: E501 line too long (85 > 79 characters) Line 1276:80: E501 line too long (84 > 79 characters) Line 1473:80: E501 line too long (80 > 79 characters) Line 1476:80: E501 line too long (82 > 79 characters) Line 1746:80: E501 line too long (83 > 79 characters)

aragilar commented 2 years ago

Sorry for not responding sooner, looks like I'll need to update the CI config so that it runs correctly. I'm slightly concerned about some of the paths changing, I'll need to do some local tests to confirm they're fine.

aragilar commented 2 years ago

Thanks @opoplawski, I've rebased (so that the CI ran) and merged this. I'll look at making a new release.