On Tue, 14 Jul 2009 16:51:53 -0400, Michael DeHaan <mdehaan(a)redhat.com>
wrote:
Here's a war everyone can get behind.
I suspect most FIXME comments in Cobbler are about things that don't
matter anymore.
Some of these things are reminders for future bits -- like 2.0 features,
that may not be implemented that we need to implement.
I'm going to be working on removing the ones that don't, and fixing the
things that need to be fixed if they do.
[mdehaan@localhost cobbler]$ grep "FIXME" *.py | wc -l
118
[mdehaan@localhost cobbler]$ pwd
/home/mdehaan/cg/cobbler/cobbler
[mdehaan@localhost cobbler]$ cd modules
[mdehaan@localhost modules]$ grep "FIXME" *.py | wc -l
23
If you'd like to help us clean up the shop, help is welcome, though I
suspect I'm the one who would know which ones are most ignorable.
Mostly it would be about removing comments so that once again we can
tell by grepping for FIXME whether the code is "done", and we can use
that for a criteria about whether 2.0 is close to done or not.
A large number of FIXME's that I see in the code are things that have
already been fixed, so hopefully this will be a short war :)
--
This message has been scanned for viruses and
dangerous content by MailScanner, and is
believed to be clean.