-
Notifications
You must be signed in to change notification settings - Fork 37
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
Multiple improvements listed below #64
Conversation
jktomer
commented
Jan 16, 2022
•
edited
Loading
edited
- pge: write temp files with .pdf extension (formerly pge: write temp files with .pdf extension #51)
- pass overwrite flag to atomic_write (formerly pass overwrite to atomic_write #56)
- utlipro: wait for page to load when clicking download link (formerly bug(ultipro_google): wait for page load after clicking download link #63) (fixes ultipro_google fails with "element not interactable" #22)
Failures are odd, look like some problem with the test environment setup? Am I able to rerun them myself? |
Yeah I don't understand the mypy cache folder issue. I deleted mine but mypy still succeeded. I don't see a way to approve you for automatic builds. |
@jbms, you might have a better idea how the testing environment is set up, and why I have the same mypy version and for me |
This is intended to address the CI failure observed in #64.
This is intended to address the CI failure observed in #64.
@jktomer, the build should succeed now after a rebase. |
Google Drive otherwise seems to get confused about their type in a way that is not fixed with a rename later
I need this because I keep my finance-dl data in Google Drive mounted with google-drive-fuse, and it doesn't support the operation atomic_write does without it. (With overwrite=True, atomicwrites does a rename; otherwise it does a link followed by an unlink. google-drive-fuse supports rename just fine, but not hard links.) The real solution is probably to fix atomicwrites to properly support all filesystems that don't support links instead of just Windows, but this is less work.
Aha—it doesn't pass, but it does fail more interestingly! Fixed; I have no idea why it passes on my machine... |
mergeable now I think? |