Repository navigation
Prompt for additional metadata during issue/pr create - #839
Conversation
|
This is ready to be taken for a spin! To test this out:
Tips:
My observations:
/cc @ampinsk |
|
I agree with those observations. I found it confusing at first, but eventually got used to it. I still wished there was a way to go back and edit but I suppose the user can select
|
probablycorey
left a comment
There was a problem hiding this comment.
This looks good and I tested it out and it worked great even on large repos!
Is it time to rename the titleBody struct now that it has so much other information? I realize this PR didn't introduce any changes in it, but I keep forgetting what that type represents now because it is so much more than titleBody now.
| return err | ||
| } | ||
| inProgress.Body = templateContents | ||
| data.Body = templateContents |
There was a problem hiding this comment.
It took me a second to figure out what data is used for. Is it a go idiom to use the name data for a var that returns a value?
There was a problem hiding this comment.
I agree that data isn't the best variable name. Basically, data here is an instance of titleBody which by now basically serves as a state object during the wizard. I'll try to rename these variables and types so it's more clear what's going on.
| } | ||
|
|
||
| confirmA, err := confirmSubmission(allowPreview) | ||
| confirmA, err := confirmSubmission(!data.HasMetadata(), true) |
There was a problem hiding this comment.
The args here make it a little confusing. Maybe pull them out as individual vars instead of passing the args inline?
@tierninho Nice find! |
|
Overall I think this is looking awesome!
Agreed this is a little awkward but I think under the circumstances and once you're used to it it's not too bad. Maybe we can update the instructions
+1 on using a spinner for these long waits! |
This matches `issue create` behavior 4b32e3f


Per discussion finalized in #768 (comment), this adds add an
Add metadataprompt toissue createandpr createwizards.Todo:
Depends on #787
Closes #768, fixes #340