Hi everyone,
here's a little heads up.
For the next five weeks, students from Freie Universität Berlin will contribute
to the Saros project.
Last week was their introduction week (you might have noticed in Gerrit the many
small patches that deal with SonarQube issues).
This is how this whole project works:
- Eventually, the students will provide patches that should be integrated into
the normal Saros code base, i.e. the master branch of the saros repository.
Those are the patches I encourage everyone to review.
- For preparing those "final" patches, the student team has a repository of its
own called "saros-swtp2016".
It's up to them whether they create patches for the normal "saros" repository
directly (as they did for the micro-patches of the introduction week), or
whether they want to play around first without the need to comply with Saros'
high standards regarding internal code quality.
This works as follows:
- The students start creating patches towards a certain end, e.g. a small
feature.
- This is software development with very little knowledge about the code
base, so many things are likely to be tried out until a viable solution has
been found.
- They will review the changes among themselves until they reach a level of
common agreement.
- Eventually, after a feature took maybe 7 patches to be complete, the
students take a step back and create a streamlined version of their
changes, maybe split into 2 patches which only reflect the effective
changes, and not every detour along the way.
(This is more or less how you might work on your own, at least that's what
I do: I create commits with a very small granularity and afterwards
reorder, change, and squash them to a much shorter series of changes.)
- The team decided to make their sandbox visible or everyone, i.e. you can
read their patches and provide comments if you like and if you can spare
the time.
- The project participants are free to incorporate or to ignore your comments
in their sandbox.
That's why non-student-project developers don't have a "Code-Review" vote
on these patches.
- Eventually, the students will need to create patches for the normal saros
repository.
Again, *those* are the patches you should focus your attention on.
We will see how this works out in the first week.
Cheers,
Franz
------------------------------------------------------------------------------
_______________________________________________
DPP-Devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/dpp-devel