mirror of
https://gitlab.kitware.com/vtk/vtk-m
synced 2024-10-05 01:49:02 +00:00
Rewrite the VTK-m CONTRIBUTING document.
This commit is contained in:
parent
6b4054c37a
commit
49b1146169
386
CONTRIBUTING.md
386
CONTRIBUTING.md
@ -1,51 +1,387 @@
|
||||
Contributing to VTK-m
|
||||
=====================
|
||||
|
||||
This page documents at a very high level how to contribute to VTK-m.
|
||||
Please check our [developer instructions][] for a more detailed guide to
|
||||
developing and contributing to the project.
|
||||
This page documents how to develop VTK-m through [Git](http://git-scm.com).
|
||||
|
||||
1. Register [GitLab Access] to create an account and select a user name.
|
||||
Git is an extremely powerful version control tool that supports many
|
||||
different "workflows" for individual development and collaboration.
|
||||
Here we document procedures used by the VTK-m development community.
|
||||
In the interest of simplicity and brevity we do *not* provide an
|
||||
explanation of why we use this approach.
|
||||
|
||||
|
||||
Setup
|
||||
-----
|
||||
|
||||
Before you begin, perform initial setup:
|
||||
|
||||
1. Register [GitLab Access][] to create an account and select a user name.
|
||||
|
||||
2. [Fork VTK-m][] into your user's namespace on GitLab.
|
||||
|
||||
3. Create a local clone of the main VTK-m repository:
|
||||
3. Follow the [download instructions](download.md#clone) to create a
|
||||
local clone of the main VTK repository:
|
||||
|
||||
$ git clone https://gitlab.kitware.com/vtk/vtk-m.git vtkm
|
||||
$ cd vtkm
|
||||
$ git clone https://gitlab.kitware.com/vtk/vtk-m.git VTK
|
||||
$ cd vtk-m
|
||||
The main repository will be configured as your `origin` remote.
|
||||
|
||||
4. Associate your GitLab fork of VTK-m VTK-m repository:
|
||||
4. (Optional but highly recommended.)
|
||||
[Register](https://open.cdash.org/register.php) with the VTK-m project
|
||||
on Kitware's CDash instance to better know how your code performs in
|
||||
regression tests. After registering and signing in, click on
|
||||
"All Dashboards" link in the upper left corner, scroll down and click
|
||||
"Subscribe to this project" on the right of VTK-m.
|
||||
|
||||
$ git remote add gitlab git@gitlab.com:username/vtk-m.git
|
||||
[GitLab Access]: https://gitlab.kitware.com/users/sign_in
|
||||
[Fork VTK-m]: https://gitlab.kitware.com/vtk/vtk-m/forks/new
|
||||
|
||||
Your fork will be configured as your `gitlab` remote.
|
||||
Workflow
|
||||
--------
|
||||
|
||||
5. Edit files and create commits (repeat as needed):
|
||||
VTK-m development uses a [branchy workflow][] based on topic branches.
|
||||
Our collaboration workflow consists of three main steps:
|
||||
|
||||
1. Local Development:
|
||||
* [Update](#update)
|
||||
* [Create a Topic](#create-a-topic)
|
||||
|
||||
2. Code Review (requires [GitLab Access][]):
|
||||
* [Share a Topic](#share-a-topic)
|
||||
* [Create a Merge Request](#create-a-merge-request)
|
||||
* [Review a Merge Request](#review-a-merge-request)
|
||||
* [Revise a Topic](#revise-a-topic)
|
||||
|
||||
3. Integrate Changes:
|
||||
* [Merge a Topic](#merge-a-topic) (requires permission in GitLab)
|
||||
|
||||
[branchy workflow]: http://public.kitware.com/Wiki/Git/Workflow/Topic
|
||||
|
||||
Update
|
||||
------
|
||||
|
||||
1. Update your local `master` branch:
|
||||
|
||||
$ git checkout master
|
||||
$ git pull
|
||||
|
||||
2. Optionally push `master` to your fork in GitLab:
|
||||
|
||||
$ git push gitlab master
|
||||
|
||||
Create a Topic
|
||||
--------------
|
||||
|
||||
All new work must be committed on topic branches.
|
||||
Name topics like you might name functions: concise but precise.
|
||||
A reader should have a general idea of the feature or fix to be developed given just the branch name.
|
||||
|
||||
1. To start a new topic branch:
|
||||
|
||||
$ git fetch origin
|
||||
|
||||
2. For new development, start the topic from `origin/master`:
|
||||
|
||||
$ git checkout -b my-topic origin/master
|
||||
|
||||
3. Edit files and create commits (repeat as needed):
|
||||
|
||||
$ edit file1 file2 file3
|
||||
$ git add file1 file2 file3
|
||||
$ git commit
|
||||
|
||||
6. Push commits in your topic branch to your fork in GitLab:
|
||||
Caveats:
|
||||
* Data files must be placed in either the root data directory, or a
|
||||
folder explicitly named 'data'. This is required as VTK-m uses Git-LFS
|
||||
to efficiently support data files.
|
||||
|
||||
|
||||
Guidelines for Commit logs
|
||||
--------------------------
|
||||
|
||||
Remember to *motivate & summarize*. When writing commit logs, make sure
|
||||
that there is enough information there for any developer to read and glean
|
||||
relevant information such as:
|
||||
|
||||
1. Is this change important and why?
|
||||
2. If addressing an issue, which issue(s)?
|
||||
3. If a new feature, why is it useful and/or necessary?
|
||||
4. Are there background references or documentation?
|
||||
|
||||
A short description of what the issue being addressed and how will go a long way
|
||||
towards making the log more readable and the software more maintainable.
|
||||
|
||||
Style guidelines for commit logs are as follows:
|
||||
|
||||
1. Separate subject from body with a blank line
|
||||
2. Limit the subject line to 60 characters
|
||||
3. Capitalize the subject line
|
||||
4. Use the imperative mood in the subject line e.g. "Refactor foo" or "Fix Issue #12322",
|
||||
instead of "Refactoring foo", or "Fixing issue #12322".
|
||||
5. Wrap the body at 80 characters
|
||||
6. Use the body to explain `what` and `why` and if applicable a brief `how`.
|
||||
|
||||
Share a Topic
|
||||
-------------
|
||||
|
||||
When a topic is ready for review and possible inclusion, share it by pushing
|
||||
to a fork of your repository in GitLab. Be sure you have registered and
|
||||
signed in for [GitLab Access][] and created your fork by visiting the main
|
||||
[VTK-m GitLab][] repository page and using the "Fork" button in the upper right.
|
||||
|
||||
[VTK-m GitLab]: https://gitlab.kitware.com/vtk/vtk-m
|
||||
|
||||
1. Checkout the topic if it is not your current branch:
|
||||
|
||||
$ git checkout my-topic
|
||||
|
||||
2. Push commits in your topic branch to your fork in GitLab:
|
||||
|
||||
$ git push gitlab HEAD
|
||||
|
||||
7. Visit your fork in GitLab, browse to the "**Merge Requests**" link on the
|
||||
left, and use the "**New Merge Request**" button in the upper right to
|
||||
create a Merge Request.
|
||||
Notes:
|
||||
* If you are revising a previously pushed topic and have rewritten the
|
||||
topic history, add `-f` or `--force` to overwrite the destination.
|
||||
|
||||
The output will include a link to the topic branch in your fork in GitLab
|
||||
and a link to a page for creating a Merge Request.
|
||||
|
||||
Create a Merge Request
|
||||
----------------------
|
||||
|
||||
Visit your fork in GitLab, browse to the "**Merge Requests**" link on the
|
||||
left, and use the "**New Merge Request**" button in the upper right to
|
||||
reach the URL printed at the end of the [previous step](#share-a-topic).
|
||||
It should be of the form:
|
||||
|
||||
https://gitlab.kitware.com/<username>/vtk-m/merge_requests/new
|
||||
|
||||
Follow these steps:
|
||||
|
||||
1. In the "**Source branch**" box select the `<username>/vtk-m` repository
|
||||
and the `my-topic` branch.
|
||||
|
||||
2. In the "**Target branch**" box select the `vtk/vtk-m` repository and
|
||||
the `master` branch. It should be the default.
|
||||
|
||||
3. Use the "**Compare branches**" button to proceed to the next page
|
||||
and fill out the merge request creation form.
|
||||
|
||||
4. In the "**Title**" field provide a one-line summary of the entire
|
||||
topic. This will become the title of the Merge Request.
|
||||
|
||||
Example Merge Request Title:
|
||||
|
||||
Add OpenMP Device Adapter
|
||||
|
||||
5. In the "**Description**" field provide a high-level description
|
||||
of the change the topic makes and any relevant information about
|
||||
how to try it.
|
||||
* Use `@username` syntax to draw attention of specific developers.
|
||||
This syntax may be used anywhere outside literal text and code
|
||||
blocks. Or, wait until the [next step](#review-a-merge-request)
|
||||
and add comments to draw attention of developers.
|
||||
* Optionally use a fenced code block with type `message` to specify
|
||||
text to be included in the generated merge commit message when the
|
||||
topic is [merged](#merge-a-topic).
|
||||
|
||||
Example Merge Request Description:
|
||||
|
||||
This branch adds a new device adapter that uses new OpenMP 4+ features
|
||||
including Task groups to better handle unbalanced and irregular domains
|
||||
|
||||
```message
|
||||
Add a OpenMP 4+ task-based device adapter.
|
||||
```
|
||||
|
||||
Cc: @user1 @user2
|
||||
|
||||
6. The "**Assign to**", "**Milestone**", and "**Labels**" fields
|
||||
may be left blank.
|
||||
|
||||
7. Use the "**Submit merge request**" button to create the merge request
|
||||
and visit its page.
|
||||
|
||||
Guidelines for Merge Requests
|
||||
-----------------------------
|
||||
|
||||
Remember to *motivate & summarize*. When creating a merge request, consider the
|
||||
reviewers and future perusers of the software. Provide enough information to motivate
|
||||
the merge request such as:
|
||||
|
||||
1. Is this merge request important and why?
|
||||
2. If addressing an issue, which issue(s)?
|
||||
3. If a new feature, why is it useful and/or necessary?
|
||||
4. Are there background references or documentation?
|
||||
|
||||
Also provide a summary statement expressing what you did and if there is a choice
|
||||
in implementation or design pattern, the rationale for choosing a certain path.
|
||||
Notable software or data features should be mentioned as well.
|
||||
|
||||
A well written merge request will motivate your reviewers, and bring them up
|
||||
to speed faster. Future software developers will be able to understand the
|
||||
reasons why something was done, and possibly avoid chasing down dead ends,
|
||||
Although it may take you a little more time to write a good merge request,
|
||||
you’ll likely see payback in faster reviews and better understood and
|
||||
maintainable software.
|
||||
|
||||
|
||||
VTK-m uses GitLab for code review and Buildbot to test proposed
|
||||
patches before they are merged.
|
||||
Review a Merge Request
|
||||
----------------------
|
||||
|
||||
Our [Issue Tracker][] is used to document feature requests and technical issues.
|
||||
Add comments mentioning specific developers using `@username` syntax to
|
||||
draw their attention and have the topic reviewed. After typing `@` and
|
||||
some text, GitLab will offer completions for developers whose real names
|
||||
or user names match.
|
||||
|
||||
Our [Wiki][] is out to propose new infrastructure designs and host other
|
||||
documentation.
|
||||
Comments use [GitLab Flavored Markdown][] for formatting. See GitLab
|
||||
documentation on [Special GitLab References][] to add links to things
|
||||
like merge requests and commits in other repositories.
|
||||
|
||||
[developer instructions]: http://m.vtk.org/index.php/Contributing_to_VTK-m
|
||||
[GitLab Access]: https://gitlab.kitware.com/users/sign_in
|
||||
[Fork VTK-m]: https://gitlab.kitware.com/vtk/vtk/fork/new
|
||||
[Issue Tracker]: https://gitlab.kitware.com/vtk/vtk-m/issues
|
||||
[Wiki]: http://m.vtk.org/
|
||||
[GitLab Flavored Markdown]: https://gitlab.kitware.com/help/markdown/markdown
|
||||
[Special GitLab References]: https://gitlab.kitware.com/help/markdown/markdown#special-gitlab-references
|
||||
|
||||
### Reviews ###
|
||||
|
||||
Reviewers may add comments providing feedback or to acknowledge their
|
||||
approval. All comments use the [GitLab Flavored Markdown][], any line of a
|
||||
comment may be exactly one of the following votes followed by nothing but
|
||||
whitespace before the end of the line:
|
||||
|
||||
* `-1` or :-1: (`:-1:`) means "The change is not ready for integration."
|
||||
* `+1` or :+1: (`:+1:`) means "The change is ready for integration."
|
||||
|
||||
These are used to inform the author that a merge srequest has been approved
|
||||
for [merging]((#merge-a-topic).
|
||||
|
||||
|
||||
#### Fetching Changes ####s
|
||||
|
||||
One may fetch the changes associated with a merge request by using
|
||||
the `git fetch` command line shown at the top of the Merge Request
|
||||
page. It is of the form:
|
||||
|
||||
$ git fetch https://gitlab.kitware.com/$username/vtk-m.git $branch
|
||||
|
||||
This updates the local `FETCH_HEAD` to refer to the branch.
|
||||
|
||||
There are a few options for checking out the changes in a work tree:
|
||||
|
||||
* One may checkout the branch:
|
||||
|
||||
$ git checkout FETCH_HEAD -b $branch
|
||||
or checkout the commit without creating a local branch:
|
||||
|
||||
$ git checkout FETCH_HEAD
|
||||
|
||||
* Or, one may cherry-pick the commits to minimize rebuild time:
|
||||
|
||||
$ git cherry-pick ..FETCH_HEAD
|
||||
|
||||
### Robot Reviews ###
|
||||
|
||||
The "Kitware Robot" automatically performs basic checks on the commits
|
||||
and adds a comment acknowledging or rejecting the topic. This will be
|
||||
repeated automatically whenever the topic is updated.
|
||||
A re-check may be explicitly requested by adding a comment with a single
|
||||
[*trailing* line](#trailing-lines):
|
||||
|
||||
Do: check
|
||||
|
||||
A topic cannot be [merged](#merge-a-topic) until the automatic review
|
||||
succeeds.
|
||||
|
||||
### Testing ###
|
||||
|
||||
VTK-m has a [buildbot](http://buildbot.net) instance watching for merge requests
|
||||
to test. Each time a merge request is updated the buildbot user (@buildbot)
|
||||
will automatically trigger a new build on all VTK-m buildbot workers. The
|
||||
buildbot user (@buildbot) will respond with a comment linking to the CDash
|
||||
results when it schedules builds.
|
||||
|
||||
The buildbot user (@buildbot) will also respond to any comment with the form:
|
||||
|
||||
Do: test
|
||||
|
||||
The `Do: test` command accepts the following arguments:
|
||||
|
||||
* `--oneshot`
|
||||
only build the *current* hash of the branch; updates will not be built
|
||||
using this command
|
||||
* `--stop`
|
||||
clear the list of commands for the merge request
|
||||
* `--superbuild`
|
||||
build the superbuilds related to the project
|
||||
* `--clear`
|
||||
clear previous commands before adding this command
|
||||
* `--regex-include <arg>` or `-i <arg>`
|
||||
only build on builders matching `<arg>` (a Python regular expression)
|
||||
* `--regex-exclude <arg>` or `-e <arg>`
|
||||
excludes builds on builders matching `<arg>` (a Python regular
|
||||
expression)
|
||||
|
||||
Multiple `Do: test` commands may be given in separate comments. Buildbot may
|
||||
skip tests for older branch updates that have not started before a test for
|
||||
a new update is requested.
|
||||
|
||||
Builder names always follow this pattern:
|
||||
|
||||
project-host-os-libtype-buildtype+feature1+feature2
|
||||
|
||||
* project: always `vtk-m`
|
||||
* host: the buildbot host
|
||||
* os: one of `windows`, `osx`, or `linux`
|
||||
* libtype: `shared` or `static`
|
||||
* buildtype: `release` or `debug`
|
||||
* feature: alphabetical list of features enabled for the build
|
||||
|
||||
|
||||
Revise a Topic
|
||||
--------------
|
||||
|
||||
If a topic is approved during GitLab review, skip to the
|
||||
[next step](#merge-a-topic). Otherwise, revise the topic
|
||||
and push it back to GitLab for another review as follows:
|
||||
|
||||
1. Checkout the topic if it is not your current branch:
|
||||
|
||||
$ git checkout my-topic
|
||||
|
||||
2. To revise the `3`rd commit back on the topic:
|
||||
|
||||
$ git rebase -i HEAD~3
|
||||
|
||||
(Substitute the correct number of commits back, as low as `1`.)
|
||||
Follow Git's interactive instructions.
|
||||
|
||||
3. Return to the [above step](#share-a-topic) to share the revised topic.
|
||||
|
||||
Merge a Topic
|
||||
-------------
|
||||
|
||||
After a topic has been reviewed and approved in a GitLab Merge Request,
|
||||
authorized developers may add a comment with a single
|
||||
[*trailing* line](#trailing-lines):
|
||||
|
||||
Do: merge
|
||||
|
||||
to ask that the change be merged into the upstream repository. By
|
||||
convention, only merge if you have recieved `+1` . Do not request a merge if
|
||||
any `-1` review comments have not been resolved.
|
||||
|
||||
### Merge Success ###
|
||||
|
||||
If the merge succeeds the topic will appear in the upstream repository
|
||||
`master` branch and the Merge Request will be closed automatically.
|
||||
|
||||
### Merge Failure ###
|
||||
|
||||
If the merge fails (likely due to a conflict), a comment will be added
|
||||
describing the failure. In the case of a conflict, fetch the latest
|
||||
upstream history and rebase on it:
|
||||
|
||||
$ git fetch origin
|
||||
$ git rebase origin/master
|
||||
|
||||
|
||||
Return to the [above step](#share-a-topic) to share the revised topic.
|
||||
|
Loading…
Reference in New Issue
Block a user