Skip to content

python3 compatable re-write (rather than just 2to3) #15

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

Open
wants to merge 25 commits into
base: master
Choose a base branch
from

Conversation

imgurbot12
Copy link

Hey there! I wanted to use this library in python3, but noticed the code could use some improvements. I was planning on using the library to generate my own ebooks for different publications online, so I decided to clean it up entirely and basically complete an entire re-write inspired by your original design.

The new code has been fully tested, and I've even shoved the results through epubcheck to ensure everything works as intended. If you like the new code design please consider merging it :)

@imgurbot12 imgurbot12 changed the title Feat/py3 python3 compatable re-write (rather than just 2to3) Jun 22, 2020
@mcepl
Copy link

mcepl commented Jan 9, 2021

Except that the tests don’t look like passing. See py3-setup-test-log.txt for complete logs.

Concerning comparing two lists? Does order matter? Shouldn't we compare just two sets?

@imgurbot12
Copy link
Author

Except that the tests don’t look like passing. See py3-setup-test-log.txt for complete logs.

Concerning comparing two lists? Does order matter? Shouldn't we compare just two sets?

interesting. I've never had an issue when running the tests directly with python3 -m pypub.tests. I've updated the test to use sets to avoid this issue however. nice catch :)

@evmer
Copy link

evmer commented Aug 16, 2022

Updates? Can we merge?

@imgurbot12
Copy link
Author

Updates? Can we merge?

I'm happy with where the PR is now, but we'd have to wait for the author to complete the merge since I obviously don't have permissions to do it on my own. Unfortunately I haven't seen any engagement from him at all so far.

@avalonv
Copy link

avalonv commented Nov 13, 2022

@imgurbot12 have you considered publishing your own fork on PyPi under a new name? Normally that might not be super polite, but I think after two years it's probably okay

@imgurbot12
Copy link
Author

@avalonv I've decided to do just that and just managed to finalize the first release. This is my first pypi published project so PRs are welcome lol.

Here's the link to the new project if anyone is interested: https://pypi.org/project/pypub3/

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.

5 participants