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

Slightly refactor launcher #221

Closed
wants to merge 86 commits into from
Closed

Slightly refactor launcher #221

wants to merge 86 commits into from

Conversation

robkam
Copy link
Member

@robkam robkam commented Nov 14, 2023

use Path more often to work across different OS
new util checkxml.py module instead of external grep
changed file name prefix in domain.py - allow dots in names and replace non-alphanumeric with tilde instead of underscore
use py7zr instead of external 7z and out of launch.py to compress.py module

robkam and others added 30 commits October 25, 2023 21:39
* fix unordered list style

- to * for bullets, same as other .md files

* fix unordered list style

* fix unordered list style

* fix link to installation instructions 

installation instructions are no longer in the readme

* + instructions for uploader

* fix link

* move info to readme.md

* Add link to IA API

The readme might be the first place somebody looks for info about Internet Archive’s API.

* add links to APIs

* Types of dump
* Robkam fix bullets (#1)

* fix unordered list style

- to * for bullets, same as other .md files

* fix unordered list style

* fix unordered list style

* fix link to installation instructions 

installation instructions are no longer in the readme

* + instructions for uploader

* fix link

* move info to readme.md

* Add link to IA API

The readme might be the first place somebody looks for info about Internet Archive’s API.

* add links to APIs

* Types of dump

* rm gui.py, mv launcher and uploader to subfolders

* Delete wikiteam3/launcher.py

* Delete wikiteam3/uploader.py

* Update __init__.py

* Update DG __init__.py

* Update L __init__.py

* Update WT3 __init__.py

* Update DG __init__.py

* Create L __init__.py

* Create L __main__.py

* Create launch.py

* Create U __init__.py

* Create U __main__.py

* Create upload.py

* Update WT3 __init__.py
* rm gui.py, mv launcher and uploader to subfolders, tweak __init__ & __main__

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* add shebang to get past CI

* [pre-commit.ci] auto fixes from pre-commit.com hooks

for more information, see https://pre-commit.ci

* revert

* rm gui.py, mv launcher and uploader to subfolders

* Delete wikiteam3/launcher.py

* Delete wikiteam3/uploader.py

* Update __init__.py

* Update DG __init__.py

* Update L __init__.py

* Update WT3 __init__.py

* Update DG __init__.py

* Create L __init__.py

* Create L __main__.py

* Create launch.py

* Create U __init__.py

* Create U __main__.py

* Create upload.py

* Update WT3 __init__.py

---------

Co-authored-by: pre-commit-ci[bot] <66853113+pre-commit-ci[bot]@users.noreply.github.com>
@robkam robkam added the enhancement New feature or request label Nov 14, 2023
# If both checks passed
if checktags and checkends:
os.chdir(Path(wikidir))
time.sleep(1)
Copy link
Member Author

@robkam robkam Nov 14, 2023

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the purpose of the delay here?

compress_images(prefix)
# End of compression section

time.sleep(1)
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

What's the purpose of the delay here?

@robkam robkam changed the title Slightly refactor launcher.py Slightly refactor launcher and uploader Nov 14, 2023
@robkam
Copy link
Member Author

robkam commented Nov 14, 2023

I have some other enhancements in mind but PR now before adding further complexity.

@robkam robkam changed the title Slightly refactor launcher and uploader Slightly refactor launcher Nov 17, 2023
@robkam robkam mentioned this pull request Nov 17, 2023
@robkam robkam closed this by deleting the head repository Jun 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants