When cloning the ScorchingShade/Encryptor repository, 32 out of 38MB of disk space used are from the venv/ directory.
Generally, it is best practice to exclude virtual environment directories from repositories, as discussed here and here. Thus, I've replaced the venv/ directory with a requirements.txt file as described here.
I also noticed that the virtual environment does not include progress, which you import Bar from. This results in an ImportError due to progress not being installed. I found that progress wasn't mentioned in the README.md.
To fix these issues, I've done the following:
Included a requirements.txt that instructs pip to install pycryptodome as well as progress
Edited README.md to use the requirements.txt, and added progress as a declared requirement
Removed the venv/ directory
This makes the setup process more flexible, and reduces the repository size from 32MB to 5.8MB.
When cloning the
ScorchingShade/Encryptorrepository, 32 out of 38MB of disk space used are from thevenv/directory.Generally, it is best practice to exclude virtual environment directories from repositories, as discussed here and here. Thus, I've replaced the
venv/directory with arequirements.txtfile as described here.I also noticed that the virtual environment does not include
progress, which you importBarfrom. This results in anImportErrordue toprogressnot being installed. I found thatprogresswasn't mentioned in theREADME.md.To fix these issues, I've done the following:
requirements.txtthat instructspipto installpycryptodomeas well asprogressREADME.mdto use therequirements.txt, and addedprogressas a declared requirementvenv/directoryThis makes the setup process more flexible, and reduces the repository size from 32MB to 5.8MB.
Let me know if anything needs changing!