Browse Source

Ussuri contrib docs community goal

This patch standardizes the CONTRIBUTING.rst file and adds the
required doc/source/contributor/contributing.rst

Swift already had a detailed CONTIRBUTING.rst and an informative
REVIEW_GUIDELINES.rst in the root of the repo. So we are also pulling
them into the contributor documentation so they can not only be easily
found in the checked repo but in the online documentation.

Change-Id: I4c84efbe50eb25ab922c9d6b69198dae341af48b
Matthew Oliver 1 year ago
committed by Tim Burke
5 changed files with 100 additions and 1 deletions
  1. +10
  2. +1
  3. +79
  4. +1
  5. +9

+ 10
- 0

@ -21,6 +21,16 @@ Swift is nothing without the community behind it. We'd love to welcome you to
our community. Come find us in #openstack-swift on freenode IRC or on the
OpenStack dev mailing list.
For general information on contributing to OpenStack, please check out the
`contributor guide <>`_ to get started.
It covers all the basics that are common to all OpenStack projects: the accounts
you need, the basics of interacting with our Gerrit review system, how we
communicate as a community, etc.
If you want more Swift related project documentation make sure you checkout
the Swift developer (contributor) documentation at
Filing a Bug

+ 1
- 1

@ -369,7 +369,7 @@ Endeavor to leave a positive or negative score on every change you review.
Use your best judgment.
A note on Swift Core Maintainers
Swift Core maintainers may provide positive reviews scores that *look*
different from your reviews - a "+2" instead of a "+1".

+ 79
- 0
doc/source/contributor/contributing.rst View File

@ -0,0 +1,79 @@
.. include:: ../../../CONTRIBUTING.rst
People working on the Swift project may be found in the
``#openstack-swift`` channel on Freenode during working hours
in their timezone. The channel is logged, so if you ask a question
when no one is around, you can check the log to see if it's been
weekly meeting
This is a Swift team meeting. The discussion in this meeting is about
all things related to the Swift project:
- time:
- agenda:
mailing list
We use the mailing list for
asynchronous discussions or to communicate with other OpenStack teams.
Use the prefix ``[swift]`` in your subject line (it's a high-volume
list, so most people use email filters).
More information about the mailing list, including how to subscribe
and read the archives, can be found at:
Contacting the Core Team
The swift-core team is an active group of contributors who are responsible
for directing and maintaining the Swift project. As a new contributor, your
interaction with this group will be mostly through code reviews, because
only members of swift-core can approve a code change to be merged into the
code repository. But the swift-core team also spend time on IRC so feel
free to drop in to ask questions or just to meet us.
.. note::
Although your contribution will require reviews by members of
swift-core, these aren't the only people whose reviews matter.
Anyone with a gerrit account can post reviews, so you can ask
other developers you know to review your code ... and you can
review theirs. (A good way to learn your way around the codebase
is to review other people's patches.)
If you're thinking, "I'm new at this, how can I possibly provide
a helpful review?", take a look at `How to Review Changes the
OpenStack Way
Or for more specifically in a Swift context read :doc:`review_guidelines`
You can learn more about the role of core reviewers in the OpenStack
governance documentation:
The membership list of swift-core is maintained in gerrit:,members
You can also find the members of the swift-core team at the Swift weekly
Getting Your Patch Merged
Understanding how reviewers review and what they look for will help getting
your code merged. See `Swift Review Guidelines <contributor/review_guidelines>`_
for how we review code.
Keep in mind that reviewers are also human; if something feels stalled, then
come and poke us on IRC or add it to our meeting agenda.
Project Team Lead Duties
All common PTL duties are enumerated in the `PTL guide

+ 1
- 0
doc/source/contributor/review_guidelines.rst View File

@ -0,0 +1 @@
.. include:: ../../../REVIEW_GUIDELINES.rst

+ 9
- 0
doc/source/index.rst View File

@ -66,6 +66,15 @@ Overview and Concepts
Contributor Documentation
.. toctree::
:maxdepth: 2
Developer Documentation