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

Feature edit request body in editor #17

Closed
wants to merge 6 commits into from

Conversation

jootd
Copy link
Contributor

@jootd jootd commented Oct 31, 2022

Works fine . with vim and neovim ( on my machine :) )

Resolves #9

@jootd jootd requested a review from mfinelli as a code owner October 31, 2022 12:18
@jootd jootd changed the title Feature edit request editor Feature edit request body in editor Oct 31, 2022
@mfinelli
Copy link
Owner

Hi, can you please add the GPL copyright notice to the top of any new files that you create? (Same request for your other PR #16)

Could you also please rebase this onto the latest master from my branch? Thanks

@jootd
Copy link
Contributor Author

jootd commented Oct 31, 2022

Ofc , I will asap

main.go Show resolved Hide resolved
edit_request_editor.go Outdated Show resolved Hide resolved
main.go Outdated Show resolved Hide resolved
@mfinelli mfinelli added the enhancement New feature or request label Oct 31, 2022
@jootd
Copy link
Contributor Author

jootd commented Oct 31, 2022

So, I need to merge master into this branch and commit ,am I right ?

@mfinelli
Copy link
Owner

It would be better if you rebased on top of master. But if you merge then I think I can still clean it up before I merge to master.

@jootd jootd requested a review from mfinelli October 31, 2022 17:52
@jootd
Copy link
Contributor Author

jootd commented Oct 31, 2022

tter if you rebased on top of master. But if you merge then I think I can still clean it up before I merge to

Done, but I'm not sure that I did it right way 😌

@mfinelli mfinelli closed this in c6a837a Nov 2, 2022
@mfinelli
Copy link
Owner

mfinelli commented Nov 2, 2022

Squashed and merged! Thanks!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Implement using $EDITOR to edit the request body
2 participants