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

Add files via upload #1

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

Conversation

mjrodri
Copy link

@mjrodri mjrodri commented Jan 29, 2024

Overview

This project aims to improve a Python script for system information gathering and reporting. The script collects various system details, such as user information, installed software, antivirus status, firewall status, etc., and outputs them to a text file.

Features

  • Consolidation of PowerShell commands into a dictionary for better manageability.
  • Adoption of consistent snake_case naming convention for variables.
  • Use of F-strings for cleaner string formatting.
  • Addition of error handling mechanisms for subprocess and file operations, enhancing script robustness.
  • Modularization of the code into reusable functions for improved readability and maintainability.
  • Inclusion of docstrings for function clarity and usage explanation.
  • Ensuring consistent code formatting, including indentation, for improved readability.

How to Use

  1. Fork the repository.
  2. Clone the forked repository to your local machine.
  3. Make necessary changes or improvements to the script.
  4. Commit your changes and push them to your fork.
  5. Create a pull request to merge your changes into the original repository.
  6. Once approved, your changes will be merged.

Contributors

Michael J. Rodriguez github.com/mjrodri
and
https://github.com/CesarIllustrious

License

This project is licensed under the MIT License.

@mjrodri
Copy link
Author

mjrodri commented Jan 29, 2024

Very cool tool. Was looking for something like to hopefully add some more value and learn how to use these frameworks. Thank you.

@CesarIllustrious
Copy link
Owner

CesarIllustrious commented Jan 29, 2024 via email

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.

2 participants