Logo Questions Linux Laravel Mysql Ubuntu Git Menu
 

Actions to do in a Pull Request with 'requested changes' to notify that changes have been done

I have created a pull request in GitHub that has been reviewed and the reviewer has requested changes. I have done the changes and added a comment in the pull request conversation with a reference to the reviewer so I expect a notification has been sent to him.

Is there any other thing I must do in GitHub to reflect that I have pushed corrections to the requested changes?

like image 881
Sergio Prats Avatar asked Jul 07 '17 07:07

Sergio Prats


People also ask

How do you request changes on a pull request?

On the pull request, click Files changed. Above the changed code, click Review changes. Type a comment summarizing your feedback on the proposed changes. Select Comment to leave general feedback without explicitly approving the changes or requesting additional changes.

What do you do once a pull request is approved?

After receiving the pull request, the project maintainer has to decide what to do. If the feature is ready to go, they can simply merge it into main and close the pull request. But, if there are problems with the proposed changes, they can post feedback in the pull request.


1 Answers

What you did is standard operating procedure when replying to a change request in a pull request. Since you pushed your changes, the branch in question has been updated. If your reviewer's GitHub is configured to not receive notifications he may not have received one, but from your point of view you have met your expectations. The other Git repositories (e.g. Bitbucket) also behave similarly to this.

like image 149
Tim Biegeleisen Avatar answered Oct 17 '22 05:10

Tim Biegeleisen