Difference between revisions of "User:Elichak"
Line 88: | Line 88: | ||
* Documenting source code | * Documenting source code | ||
** Starting off with: | ** Starting off with: | ||
− | *** svn.pl and makewrapper.pl. - <b>DONE</b> | + | *** svn.pl and makewrapper.pl. - <b>DONE</b> - [[User:RichardChu|Richard Chu]] recommended some minor changes |
*** Comments about svn.pl | *** Comments about svn.pl | ||
**** For <b>Update</b> subroutine | **** For <b>Update</b> subroutine | ||
Line 94: | Line 94: | ||
***** Currently, subroutine only allows these revision values: HEAD and numbers | ***** Currently, subroutine only allows these revision values: HEAD and numbers | ||
***** Currently you allow svn update command to perform even if file is not defined, shouldn't it disallow svn update to perform if file is undefined? | ***** Currently you allow svn update command to perform even if file is not defined, shouldn't it disallow svn update to perform if file is undefined? | ||
+ | *** [[User:RichardChu|Richard Chu]]: | ||
+ | **** The users should probably be able to use those revision keywords. And it is a really minor change to allow that subroutine to allow those keywords. | ||
+ | **** I'm pretty sure the svn update command will perform even though no file is defined. | ||
** Then, files in the Test directory: - <b>IN PROGRESS</b> | ** Then, files in the Test directory: - <b>IN PROGRESS</b> | ||
*** Test.pm, TestCase.pm, TestSuite.pm, TestResult.pm, and TestRunner.pm | *** Test.pm, TestCase.pm, TestSuite.pm, TestResult.pm, and TestRunner.pm |
Revision as of 16:33, 10 December 2006
Prior to this course, I've never been involved with open source projects. This course has opened up my eyes to the Mozilla community and I'm really impressed with the work that the Mozilla community members have done in their spare time. By far, this is one of the most interesting courses I've ever taken at Seneca and I can't express greater gratitude to be given the opportunity to be part of this fascinating project.
|