Skip to content

fix(prompts): add : to the end of input prompts#2857

Merged
olimorris merged 1 commit into
olimorris:mainfrom
AmadeusK525:fix/rejection-prompt
Apr 7, 2026
Merged

fix(prompts): add : to the end of input prompts#2857
olimorris merged 1 commit into
olimorris:mainfrom
AmadeusK525:fix/rejection-prompt

Conversation

@AmadeusK525

@AmadeusK525 AmadeusK525 commented Mar 5, 2026

Copy link
Copy Markdown
Contributor

Description

Of course, this:

Rejection reason: this is my rejection reason

is better than this:

Rejection reasonthis is my rejection reason

AI Usage

N/A

Related Issue(s)

N/A

Screenshots

Checklist

  • I've read the contributing guidelines and have adhered to them in this PR
  • I confirm that this PR has been majority created by me, and not AI (unless stated in the "AI Usage" section above)
  • I've run make all to ensure docs are generated, tests pass and StyLua has formatted the code
  • (optional) I've added test coverage for this fix/feature
  • (optional) I've updated the README and/or relevant docs pages

@olimorris

Copy link
Copy Markdown
Owner

Sorry just seen this now. I tend to ignore PRs that have a fail next to them. In this case, you've not folllowed the PR template. Once fixed, happy to merge this

@olimorris

Copy link
Copy Markdown
Owner

Will be happy to merge this when the conflicts have been resolved

Of course, this:
> Rejection reason: this is my rejection reason

is better than this:
> Rejection reasonthis is my rejection reason
@AmadeusK525 AmadeusK525 force-pushed the fix/rejection-prompt branch from e687863 to e3985ad Compare April 7, 2026 13:57
@AmadeusK525

Copy link
Copy Markdown
Contributor Author

Done @olimorris, sorry for taking a while, I've been switching jobs, which implies switching computers and everything's been messy on my end 😅

@olimorris olimorris merged commit 03b4e4f into olimorris:main Apr 7, 2026
6 checks passed
@AmadeusK525 AmadeusK525 deleted the fix/rejection-prompt branch April 8, 2026 18:24
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