[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: CVS files merging procedure
From: |
Jim Hyslop |
Subject: |
Re: CVS files merging procedure |
Date: |
Wed, 14 Dec 2005 10:43:06 -0500 |
User-agent: |
Mozilla Thunderbird 1.0.6 (Windows/20050716) |
-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1
abhay mehta wrote:
> Sir,
(side note: "Sir" is an old-fashioned salutation, which is generally
frowned upon these days because it excludes women - just thought you
should know.)
> Based on feedback from Jim, Russ Sherk and others, we reviewed our
> SCM plan. However based on our requirements, we need an
> Administrator/ Reviewer to review the code before it goes to
> repository.
>
> This is very much required as we have number of programmers, who
> don't know the whole system and so the implications/ side-effects of
> their changes in other part of the system.
You still need to modify your engineering practises. This is not an SCM
problem, it is a software engineering problem.
No one person can ever completely understand a system. Furthermore, what
will you do when this person leaves the company? You're screwed.
This is why you write unit tests. When programmers make their changes,
they run the tests. If the tests pass, they check in their code. If the
tests fail, the programmers go back and fix up what they just broke.
If the tests pass, but the programmer has still introduced a problem,
then your first task is to fix the tests so they are more comprehensive.
- --
Jim Hyslop
Dreampossible: Better software. Simply. http://www.dreampossible.ca
Consulting * Mentoring * Training in
C/C++ * OOD * SW Development & Practices * Version Management
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.2 (MingW32)
Comment: Using GnuPG with Thunderbird - http://enigmail.mozdev.org
iD8DBQFDoD2KLdDyDwyJw+MRAqycAJ9Si3PysLsR5GWZszm5GOz5zXpUnQCg8iCT
qVgM4aTl3kaHb5NIcdcpU+g=
=M3dw
-----END PGP SIGNATURE-----