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

Dev #7

Closed
wants to merge 4 commits into from
Closed

Dev #7

wants to merge 4 commits into from

Conversation

dr8co
Copy link
Owner

@dr8co dr8co commented Nov 23, 2023

No description provided.

This change streamlines the process of detecting reused passwords. A map has been introduced to store the count of sites where the passwords are being reused. Reused passwords are now printed in descending order of counts, providing more valuable information to users about which password has the most reuse.

It also adds comments to clarify the purpose of the changes in the code. This makes the code more maintainable and accessible to other developers.
The workflow file 'cmake-multi-platform.yml' for multi-platform CMake builds has been modified to simplify and improve the BLAKE3 installation process. Instead of directly cloning and building BLAKE3 inside the job, an install script was incorporated which is called based on the operating system detected. This implementation ensures better OS compatibility and enhances the overall clarity and maintainability of the code.
The renaming of 'install_blake3.sh' to 'install-blake3.sh' complies with standard Unix-like operating systems naming conventions, enhancing cross-platform compatibility. It increases the readability, uniformity and makes the script name easier to type and remember.
The scripts 'install-blake3.sh' and 'buildscript.sh' have been moved to the 'scripts' directory. This provides better organization and clarity of the project structure, making the scripts easier to locate and manage.
@dr8co dr8co closed this Nov 23, 2023
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.

1 participant