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

Update versioneer to 0.21 #36

Closed
wants to merge 1 commit into from

Conversation

opoplawski
Copy link
Contributor

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

@pep8speaks
Copy link

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
Copy link
Owner

aragilar commented Dec 7, 2021

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 aragilar mentioned this pull request Jan 7, 2022
bors bot added a commit that referenced this pull request Jan 7, 2022
38: Update versioneer to 0.21 r=aragilar a=aragilar

Rebased version of #36.

Co-authored-by: Orion Poplawski <[email protected]>
@aragilar
Copy link
Owner

aragilar commented Jan 7, 2022

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

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.

3 participants