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 support for flags and .env #21

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

Conversation

Itezaz-ul-Hassan
Copy link

In the latest update, several flags have been introduced to enhance the parsing capability and facilitate future usage of the program. Additionally, to address security concerns and adhere to the standard practice of storing API Keys, the program now supports the use of .env files instead of openai_key.txt. The program will now prompt the user for confirmation before applying any changes, and if the user inputs 'y', the changes will be made and the process will be repeated. Conversely, if the user inputs 'n', the program will terminate. However, when the -y or --yes flag is used, the program will not prompt the user for confirmation at each step, and the changes will be made continuously until the script runs successfully.

@biobootloader
Copy link
Owner

Thank you, this is good. A couple other PRs have now merged that also added .env support the api key and user confirmation for changes, but this still adds useful new flags. I need to think a bit more about the interface before coming back to this and merging. I'll get to it soon!

@Itezaz-ul-Hassan
Copy link
Author

Thank you, this is good. A couple other PRs have now merged that also added .env support the api key and user confirmation for changes, but this still adds useful new flags. I need to think a bit more about the interface before coming back to this and merging. I'll get to it soon!

Sure

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