opendev/infra-manual
Code Issues Proposed changes

Merge "Add section about cross-project dependencies"

Browse Source
This commit is contained in:
Jenkins 2014-12-02 18:51:12 +00:00 committed by Gerrit Code Review
commit 7fcf346fa6
1 changed files with 22 additions and 0 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

22
doc/source/developers.rst
View File

@ -407,6 +407,28 @@ want the rebase behavior in git review. If you would rather postpone
resolving merge conflicts you can use git review ``-R`` as the last step
above.
Cross-Project Dependencies
--------------------------
If your change has a dependency on a change outside of a project, like
a change for another project or some manual setup, here are some best
practices on marking those changes so that they merge at the right
time.
* As a developer, mark your change with the "Work in Progress" label
until the dependencies are in.
* Add this header line to the commit message of a change which depends
on a change in another project::
Depends-On: <patch-url>
* Use the same topic for all changes. This allows to easily find all
changes across repositories.
* A core reviewer might block an important change with a -2 so that it
does not get merged accidentally.
Code Review
===========