Page MenuHomekolab.org

A contributor guide to Kolab
ClosedPublic

Authored by vanmeeuwen on Feb 22 2016, 12:08 PM.

Diff Detail

Repository
rD docs
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.

Event Timeline

vanmeeuwen updated this revision to Diff 170.Feb 22 2016, 12:08 PM
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.
vanmeeuwen updated this revision to Diff 198.Mar 8 2016, 2:54 PM
  • An initial version of a guide for contributors
vanmeeuwen updated this revision to Diff 199.Mar 8 2016, 2:55 PM
  • 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.
petersen added inline comments.Mar 10 2016, 7:46 AM
source/contributor-guide/bug-reporting-structured.rst
94

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

source/contributor-guide/feature-requests/index.rst
59

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
73

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

vanmeeuwen planned changes to this revision.Mar 10 2016, 2:53 PM
vanmeeuwen added inline comments.
source/contributor-guide/bug-reporting-structured.rst
94

Yup.

source/contributor-guide/feature-requests/index.rst
59

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
73

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.

vanmeeuwen updated this revision to Diff 204.Mar 11 2016, 8:49 AM
  • Add span.green
  • Provide these as macros
  • Use the macros
  • Use macros
  • Provide even more macros
  • Use macros
vanmeeuwen planned changes to this revision.Mar 11 2016, 8:49 AM
vanmeeuwen planned changes to this revision.Mar 11 2016, 8:58 AM
vanmeeuwen updated this revision to Diff 206.Mar 11 2016, 9:06 AM
  • 'be very costly exercise' -> 'be a very costly exercise'
  • construction -> elaboration, additional notes
  • Use macro
vanmeeuwen updated this revision to Diff 207.Mar 11 2016, 9:08 AM
  • Remove merge leftover
seigo accepted this revision.Mar 11 2016, 10:14 AM
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.