Page MenuHomePhorge

A contributor guide to Kolab
ClosedPublic

Authored by vanmeeuwen on Feb 22 2016, 12:08 PM.
Referenced Files
Unknown Object (File)
Sun, Apr 21, 5:19 PM
Unknown Object (File)
Fri, Apr 19, 9:55 PM
Unknown Object (File)
Fri, Apr 12, 11:39 AM
Unknown Object (File)
Wed, Apr 10, 6:21 AM
Unknown Object (File)
Sun, Apr 7, 10:15 PM
Unknown Object (File)
Sat, Mar 30, 3:19 AM
Unknown Object (File)
Thu, Mar 28, 10:50 PM
Unknown Object (File)
Thu, Mar 28, 9:42 AM

Diff Detail

Repository
rD docs
Branch
dev/T1037
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 2814
Build 2742: arc lint + arc unit

Event Timeline

vanmeeuwen retitled this revision from to Add a first iteration of development process documentation that is already outdated..
vanmeeuwen updated this object.
vanmeeuwen edited the test plan for this revision. (Show Details)
vanmeeuwen added subscribers: petersen, seigo.
  • An initial version of a guide for contributors
  • An initial version of a guide for contributors
  • Remove process.rst
vanmeeuwen retitled this revision from Add a first iteration of development process documentation that is already outdated. to A contributor guide to Kolab.Mar 8 2016, 4:03 PM
vanmeeuwen added a project: Documentation.
source/contributor-guide/bug-reporting-structured.rst
93 ↗(On Diff #199)

"..can be 'a' very costly.."?

source/contributor-guide/feature-requests/index.rst
58 ↗(On Diff #199)

According to the bullet points under this, it is actually "before the Elaboration phase" rather than the Construction phase.

In teh elaboration phase we decide how to change the world with this task, where as the Construction phase is pure hardhats, bricks and mortar.

source/contributor-guide/getting-started.rst
72 ↗(On Diff #199)

Perhaps not "a lot" more. I think it is an excellent definition you put up here.

vanmeeuwen added inline comments.
source/contributor-guide/bug-reporting-structured.rst
93 ↗(On Diff #199)

Yup.

source/contributor-guide/feature-requests/index.rst
58 ↗(On Diff #199)

OK, I'll change this.

I like the analogous terminology used, I'll put that in as a note with the to-be-authored list below.

source/contributor-guide/getting-started.rst
72 ↗(On Diff #199)

I meant to say more about working with a sprint in Phabricator specifically -- so;

  • the workboard with Backlog, Doing, Review, Done,
  • Task status changes when moving to Done (basically release manager / merge window / ... setting the status to Resolved after retrospective),
  • A differential being submitted for Review moving the task associated with it to the Review column,
  • ...

That sort of thing.

  • Add span.green
  • Provide these as macros
  • Use the macros
  • Use macros
  • Provide even more macros
  • Use macros
  • 'be very costly exercise' -> 'be a very costly exercise'
  • construction -> elaboration, additional notes
  • Use macro
seigo added a reviewer: seigo.

A really, really good step forward. Eventually we will want to add an introductory summary to give newcomers an overview of the whole process and how all the pieces fit together (the main Contributor page can / ought to become this imho), but getting the individual topics all covered first is probably the first thing to complete .. I really like how this already clearly describes so much about our workflow. Will have to blog/G+/etc the shit out of this once it is pushed ;)

This revision is now accepted and ready to land.Mar 11 2016, 10:14 AM
This revision was automatically updated to reflect the committed changes.