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

looking for ways to clean it up or improve its efficiency, I could sugg… #1

Open
wants to merge 1 commit into
base: develop
Choose a base branch
from

Conversation

brokep
Copy link

@brokep brokep commented Jun 27, 2023

looking for ways to clean it up or improve its efficiency, I could suggest the following:

Define the path to the config_files directory once at the top of your script, reducing the need to repeatedly construct it.
Check the existence of the config_files directory at the start of the script. This way, you can prevent generating certificates if the directory does not exist.
If you are not interested in retaining the standard output, redirect it to /dev/null to reduce unnecessary terminal output.

…est the following:

    Define the path to the config_files directory once at the top of your script, reducing the need to repeatedly construct it.
    Check the existence of the config_files directory at the start of the script. This way, you can prevent generating certificates if the directory does not exist.
    If you are not interested in retaining the standard output, redirect it to /dev/null to reduce unnecessary terminal output.
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