Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@Yuuki77
Copy link
Contributor

@Yuuki77 Yuuki77 commented Sep 28, 2020

Fix #517

It is now possible to comment on an issue from the editor.

gh issue comment <url> or <issue number>

How to test

1 . create a issue
2.` gh comment issue
3 add the body from editor
4 submit the comment
5 test same for url

6 please test if gh comment issue <issue number> -b "your comment" work as well
7. please check if the cancel works as well.

image

This is my first pr to this repo. If you have any comments, please let me know 😄

@@ -0,0 +1,117 @@
package comment
Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

will add more tests

@vilmibm
Copy link
Contributor

vilmibm commented Sep 29, 2020

Thanks for this!

As per our CONTRIBUTING docs we prefer it if people check in with us about adding features not marked as help-wanted; in this instance we hadn't finalized what we wanted the design for this to look like and weren't ready for a PR.

That said, this is a valid approach and one we were considering; we'd just like to take the time to ensure this is the direction we want to go in with commenting. You've done good work and we will circle back to this PR once we've had a chance to discuss internally. cc @ampinsk

@Yuuki77
Copy link
Contributor Author

Yuuki77 commented Sep 30, 2020

@vilmibm I am sorry that I missed the help-wanted thing.
Yeah Please let me know how this goes 😄
I'd be happy to contribute in any form

@maisiliym

This comment has been minimized.

@mislav
Copy link
Contributor

mislav commented Nov 19, 2020

@Yuuki77 Thank you for your contribution! At a quick glance, the code looks pretty solid to me.

My apologies about the confusion about whether we want to accept outside contributions for this feature. At the time that you opened this PR, we haven't decided on that yet. Starting from this week, we are starting development of this feature, and your PR is going to be considered and potentially incorporated in our work. Please have patience with us while we figure out exactly how we want the commands and flags to be named. 🙇

@mislav
Copy link
Contributor

mislav commented Nov 19, 2020

@li-maisiliym-xyz We generally try to welcome constructive criticism of our code or practices, but I can't see how your criticism is constructive. Your tone sounds hostile to our developers, and I've hidden it per the "insulting/derogatory comments" clause of our Code of Conduct.

@Yuuki77
Copy link
Contributor Author

Yuuki77 commented Nov 19, 2020

@mislav Thank you for the updates! 😄
yeah I am waiting for you guys decisions!

@maisiliym

This comment has been minimized.

@samcoe samcoe mentioned this pull request Dec 2, 2020
@Yuuki77 Yuuki77 closed this Dec 3, 2020
@Brazuca228

This comment was marked as spam.

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.

Comment on issues and pull requests from gh

5 participants