19

Sorry if the question is trivial, but I couldn't find an answer.

I sent a pull request to a project. Someone commented something about it.

What would be the correct way to reply to his/her comment? I can simply add a comment of my own to the page. But is this the right way? Will he/she be 'notified' (if this even exists in Github)?

1 Answer 1

12

Just add a comment of your own to the page. If their comment is in the diff (i.e. the "Files changed" tab), you can respond inline next to their original comment.

If you mention them by name (using @username), they should also get a notification telling them that someone has mentioned them.

7
  • I see. Is it cool to mention (@) the guy I'm responding to if the discussion is mostly between the two of us anyway? Because I'd like him to know I've fixed the PR so he can look at it again. If I don't use @, will he know about this only when deciding to browse to the PR?
    – Aviv Cohn
    Jan 3, 2016 at 0:40
  • I don't see any harm in mentioning him specifically. Whether he gets notified when you update the pull request depends on his notification settings. This is true for mentions also, but I would imagine that most people tend to leave that setting turned on. Jan 3, 2016 at 0:43
  • 1
    In conversations over PRs, I usually don't see use of @. So I'd like to use it to get a response sooner, but don't want to seem rude (again excuse me for the beginner question). So in your opinion using @ after updating a PR to notify the collaborator is fine?
    – Aviv Cohn
    Jan 3, 2016 at 0:48
  • 1
    Where I work now, and where I've worked in the past, we've used the @ mention to get someone's attention when a further action is needed on a pull request. This sounds like exactly what you want. Jan 3, 2016 at 0:49
  • BTW I'm talking open open source projects, if this matters. Also, what is your opinion about 'pinging' someone with @ after a while the PR didn't get attention from a collaborator?
    – Aviv Cohn
    Jan 3, 2016 at 0:53

Your Answer

By clicking “Post Your Answer”, you agree to our terms of service, privacy policy and cookie policy

Not the answer you're looking for? Browse other questions tagged or ask your own question.