Highlighted the assignee, maintainer changes, and mentioned LAMMPS collaborator

This commit is contained in:
Stefan Paquay
2017-01-04 17:28:22 +01:00
parent 933b288ce9
commit 0bcbcca140
3 changed files with 38 additions and 29 deletions

Binary file not shown.

Before

Width:  |  Height:  |  Size: 100 KiB

After

Width:  |  Height:  |  Size: 99 KiB

Binary file not shown.

After

Width:  |  Height:  |  Size: 13 KiB

View File

@ -120,7 +120,7 @@ commit` to finalize the commit. If you find doing this on the command line too
tedious, consider using a GUI, the one included in git distributions written in tedious, consider using a GUI, the one included in git distributions written in
Tk, i.e. use `git gui`. Tk, i.e. use `git gui`.
After adding all files, the change can be commited with some useful message After adding all files, the change can be committed with some useful message
that explains the change. that explains the change.
$ git commit -m 'Finally updated the github tutorial' :pre $ git commit -m 'Finally updated the github tutorial' :pre
@ -172,17 +172,31 @@ request" button, see image.
Before creating the pull request, make sure the short title is accurate Before creating the pull request, make sure the short title is accurate
and add a comment with details about your pull request. and add a comment with details about your pull request.
I guess here you write what your modifications do and why they should I guess here you write what your modifications do and why they should
be incorporated upstream. After that, click the "Create pull request" be incorporated upstream.
button, see image below.
Now just write some nice comments, click on "Create pull request". Note the checkbox that says "Allow edits from maintainers".
This is checked by default checkbox (although in my version of Firefox, only the checkmark is visible):
:c,image(JPG/tutorial_edits_maintainers.png)
If it is checked, maintainers can immediately add their own edits to the pull request.
This helps the inclusion of your branch significantly, as simple/trivial changes can be made by the maintainers directly.
The alternative would be that they make changes on their own version of the branch and file a reverse pull request to you.
Just leave this box checked unless you have a very good reason not to.
Now just write some nice comments and click on "Create pull request".
:c,image(JPG/tutorial_create_new_pull_request2.png) :c,image(JPG/tutorial_create_new_pull_request2.png)
:line
[After filing a pull request] [After filing a pull request]
You will notice some that after filing the pull request, NOTE: When you submit a pull request (or ask for a pull request), you will receive an
some checks are performed automatically: invitation to become a LAMMPS project collaborator. This will simplify certain
administrative tasks and will probably speed up the merging of your feature.
You will notice that after filing the pull request, some checks are performed automatically:
:c,image(JPG/tutorial_automated_checks.png) :c,image(JPG/tutorial_automated_checks.png)
@ -190,19 +204,22 @@ If all is fine, you will see this:
:c,image(JPG/tutorial_automated_checks_passed.png) :c,image(JPG/tutorial_automated_checks_passed.png)
A few further interesting things (can) happen to pull requests before A few further interesting things (can) happen to pull requests before they are included.
they are included.
[Additional changes]
First of all, any additional changes you push into your branch in your First of all, any additional changes you push into your branch in your
repository will automatically become part of the pull request: repository will automatically become part of the pull request:
:c,image(JPG/tutorial_additional_changes.png) :c,image(JPG/tutorial_additional_changes.png)
This is useful because it allows you to add parts that should be part of This means you can add changes that should be part of the feature after filing the pull request,
the feature after filing the pull request in case you have forgotten them, which is useful in case you have forgotten them,
or after a developer has ruled that something needs to change. or if a developer has ruled that something needs to change before the feature can be accepted upstream.
After each push, the automated checks are run again. After each push, the automated checks are run again.
[Assignees]
There is also now an assignee label. If the request has not been reviewed There is also now an assignee label. If the request has not been reviewed
by any developer yet, it is not assigned to anyone. After revision, a developer by any developer yet, it is not assigned to anyone. After revision, a developer
can choose to assign it to either a) you, b) a LAMMPS developer can choose to assign it to either a) you, b) a LAMMPS developer
@ -211,33 +228,25 @@ can choose to assign it to either a) you, b) a LAMMPS developer
Case a) happens if changes are required on your part. Case a) happens if changes are required on your part.
Case b) means that at the moment, it is being tested and reviewed by a LAMMPS developer. Case b) means that at the moment, it is being tested and reviewed by a LAMMPS developer.
After review, the developer can choose to implement changes or suggest them to you. After review, the developer can choose to implement changes or suggest them to you.
Case c) means that the pull request has been assigned to the lead developer, and means Case c) means that the pull request has been assigned to the lead developer Steve Plimpton,
it is considered ready for merging. and means it is considered ready for merging.
NOTE: When you submit a pull request (or ask for a pull request), you will receive an In this case, Axel assigned the tutorial to Steve:
invitation to become a LAMMPS project collaborator. This will simplify certain
administrative tasks and will probably speed up the merging of your feature.
If you allow LAMMPS maintainers to push into your branch, they can, at the time
of review, push changes they deem necessary into your branch. This can significantly
speed up the procedure if only small fixes are required. In this case, akohlmey and
rbberger made use of this to add some changes to improve this feature even before it
is merged into the main distribution:
:c,image(JPG/tutorial_changes_others.png)
After the developers review the pull request, they typically assign
it to someone, the {assignee}. For this pull request, the assignee is
sjplimp.
:c,image(JPG/tutorial_steve_assignee.png) :c,image(JPG/tutorial_steve_assignee.png)
[Edits from LAMMPS maintainers]
If you allowed edits from maintainers (the default), any LAMMPS maintainer can add changes to your pull request.
In this case, both Axel and Richard made changes to the tutorial:
:c,image(JPG/tutorial_changes_others.png)
:line :line
[After a merge] [After a merge]
When everything is fine the feature branch is merged into the LAMMPS When everything is fine, the feature branch is merged into the master branch.
repositories:
:c,image(JPG/tutorial_merged.png) :c,image(JPG/tutorial_merged.png)