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

Don't override user provided title and body when --fill is used #1994

Open
wants to merge 1 commit into
base: trunk
from

Conversation

@msfjarvis
Copy link

msfjarvis commented Sep 27, 2020

Fixes #1400

I'm not a Go developer and couldn't find any pre-existing tests for the --fill flag so haven't added any in this PR. Existing tests (ran using make test) still pass.

I created this PR using my generated binary with this command: ./bin/gh pr create -t "Don't override user provided title and body when --fill is used" --fill -d so I guess that is a test in itself :D

@msfjarvis msfjarvis marked this pull request as ready for review Sep 27, 2020
Signed-off-by: Harsh Shandilya <me@msfjarvis.dev>
@msfjarvis msfjarvis force-pushed the msfjarvis:pr-create-fill-priority branch from 60cfee8 to b16308c Sep 27, 2020
@bigcook12345
Copy link

bigcook12345 commented Sep 27, 2020

That's why I corrected my typo. No harm, hopefully it helped out

@samcoe samcoe added the community label Sep 28, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

3 participants
You can’t perform that action at this time.