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

renaming VAR for multiline option (-b | --big-prompt) option and its … #136

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

Conversation

JulianvonGebhardi
Copy link

…fix to prevent error: UNKNOWN PARAMETER

I found a little issue with the multiline option. The option -b (--big-prompt) produced the error: "Unknown parameter". I quickly fixed it and renamed the var to match the options in your README.md for clarification.

And props! I really it!

@JulianvonGebhardi
Copy link
Author

Since I just found the previous commit related to this. Another soltuion might be:

  • keeping the naming "multi_line_prompt" and using -M to distinguish the -m (model) option.
    But it needed a fix anyway since the code did not comply with the stated option -b | --big-prompt in the README file. Now it works as intended.

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