Skip to content
This repository has been archived by the owner on Jan 23, 2022. It is now read-only.

Updated the documentation, cleaned the code structure, added commented docs, removed some dependencies #17

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

Conversation

r1shavd
Copy link

@r1shavd r1shavd commented May 4, 2021

In this update, I did the following thing mostly :

  1. Firstly, I removed the pycache/ folder from the repo, which was consuming waste storage + also ignored all kind of the python cache directories in all major roots.
  2. Made the code clean and error free.
  3. Added the function of quiting the script immediately when the user presses the CTRL+C.
  4. Added the commented docs, making things easy for the readers of the code.
    So, the overall change is made to the Cryptography/ directory scripts (tools).
    Hope so you review my contribution to this repository, and hope so you merge it soon.

r1shavd added 2 commits May 3, 2021 21:37
…_pycache__ directory. Previously, it was written for all the files inside that directory to be ignored, but now it is changed to the entire directory itself will be ignored. Although, the pycache folder is a waste folder, which has no need, but still makes the crowd. So, better to remove it. Also, added the code for ignoring the other __pycache__ folders inside the other directories.
Changes made in this commit :
1. Made the code clean and error free.
2. Added the function of quiting the script immediately when the user presses the CTRL+C.
3. Added the commented docs, making things easy for the readers of the code.
@Naategh
Copy link
Owner

Naategh commented Jun 26, 2021

hey @RDofficial,
Thanks for your contribution, I think your changes contain a bit of useless information like the description of the contributors and modification times, If you remove that information I'll merge the pull.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants