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

Adds helpers for working with prompts #87

Merged
merged 5 commits into from
Jul 13, 2024

Conversation

jaresty
Copy link
Collaborator

@jaresty jaresty commented Jul 10, 2024

  • Sometimes it's useful to set the prompt to your clipboard or view in the browser

- Sometimes it's useful to set the prompt to your clipboard or view in the browser
@jaresty jaresty force-pushed the add-helpers-for-working-with-prompts branch from 60ea9d5 to 67cc733 Compare July 11, 2024 05:12
@C-Loftus
Copy link
Owner

Ok I am fine with this now. I put this in gpt-beta since I am thinking this might be condensed into the grammar at some point but not sure what that will exactly look like now. I am using model pass <prompt> so it fits the syntax better. model prompt is a bit confusing imo. model pass signifies we are doing a noop where the source is the prompt itself.

Added some comments. Let's try to always get comments on the talon files whenever we add something.

@C-Loftus
Copy link
Owner

Also moves the code that adds the snip modifier prompt before the code that does the passing. This doesn't change any behavior atm, but in general as a best practice we should be checking special cases after the prompt is finalized by checking modifiers.

@jaresty
Copy link
Collaborator Author

jaresty commented Jul 13, 2024

Looks good to me!

@C-Loftus C-Loftus merged commit 9ad1833 into main Jul 13, 2024
3 checks passed
@C-Loftus C-Loftus deleted the add-helpers-for-working-with-prompts branch July 13, 2024 18:47
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