-
Notifications
You must be signed in to change notification settings - Fork 2.2k
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
Add 'hub comment' command #324
Conversation
Usage examples: hub comment "Awesome commit, man!" hub comment 4c46aff "Awesome commit, man!" If no commit specified, it will use the current commit.
Not sold on this yet. GH comments are useful, but I prefer when they reference a certain line, and your feature doesn't support that, and even if it did, it would be super-awkward to specify it on the command line. Also, a lot of people writing comments use emoji, @-mentions, and embed images by drag-drop. The user experience with all of these features is superior to that of command-line/text editor. For me it makes sense to author pull requests in vim because that's longer passages of text and often includes code as well. GitHub commit comments, in constrast, are smaller and I don't see myself submitting them via command line. |
I agree that it would feel awkward to reference a certain line from terminal - I'm considering trying to write a vim plugin for that scenario and see how that feels. Use-case for this feature - you pull some code, take a look at whatchanged, make a couple of quick comments like "this needs tests", "great job", "you forgot about x". But of course, you're right, I don't see anyone writing comments only using I can certainly respect not wanting to add something that you feel won't be very useful (even if I disagree 😄). Not overburdening something with too many features is a good rule for any software. |
I was thinking of breaking hub into multiple mini-executables which would collaborate together to cover the current feature-set. Using those, you could write a custom executable that reads information from hub about the current repo that would allow you to implement features like this for yourself and potentially share with other people (kinda as plugins). |
I'm also 👎 on making it easier to comment on specific commits. Commit On Fri, May 10, 2013 at 5:23 PM, Mislav Marohnić
Engineer @ GitHub |
Sure, thanks for taking the time to consider this 👍 |
Usage examples:
hub comment "Awesome commit, man!"
hub comment 4c46aff "Awesome commit, man!"
If no commit specified, it will use the current commit.
It can still be infinitely improved. Wanted to get feedback first,
before continuing ahead.