User:MarkAHershberger/Weekly reports/2011-01-10
Achievemants
[edit]- Code Review (1.17 is in sight!) [Partial]
- Fix tests so they can run anywhere [Not Done]
- Update mw:Unit Testing with some policy about how to write tests. [Not Done]
Bonus
[edit]Had a good discussion with Robla about The Parser and how it has become a boogey man. After some reflection, I think people's timidity around the parser is causing some serious stagnation. People are afraid of it becuase they don't understnad it. Work needs to be done to make it understandable and easy (at least easier) to work with with,
Objectives
[edit]- Code Review (Help Robla clear the queue for un-reviewed revisions by giving them assignments)
- Fix tests so they can run anywhere (Maybe ask YetAnotherX to see if he can handle this)
- Update mw:Unit Testing with some policy about how to write tests.
Daily Log
[edit]- Made a contribution to the forever recurring discussion about the parser.
- Started looking at revs in Parser code that don't have any comments. r71966, r70949, r70900, r70822, r70524, r68374, r68342
- Code Review:
r70949… spawned an IRC conversation in which I learned a bit about user languages and whatnot. (also see r43077)r71966… commit comment points toisAlwaysKnown()and says it takes care of NS_SPECIAL cases that exist. Checked and varified.r70900… meant trudging through the code and comparing this rev with r72481 to figure out if there was a problem.
- Code Review
- Trying to understand Preprocessor_DOM