?

Log in

No account? Create an account
September 7th, 2002 - LiveJournal Development [entries|archive|friends|userinfo]
LiveJournal Development

[ userinfo | livejournal userinfo ]
[ archive | journal archive ]

September 7th, 2002

BORED? Review some patches! [Sep. 7th, 2002|05:56 am]
LiveJournal Development

lj_dev

[mart]

Non-trivial patches on Zilla will invariably be ignored until someone gives them some peer-review. If you've got a spare half-hour you could spend it reviewing patches submitted by your peers!

When reviewing a patch, look out for bugs such as logic reversal, code regression (that is, when the patch breaks some other bit of code), security problems (such as introducing user-supplied data into SQL queries unescaped) and any other issues you can see which would prevent the patch from committing to CVS and performing the desired function.

If it is your opinion that the code is correct, add the 'reviewed' keyword to the bug, and also mark the particular patch file(s) you reviewed with the 'reviewed' checkbox, so we can see what you reviewed. When doing the first of these, also leave a comment stating that you have reviewed the patch and any comments you may have about it. This ensures that the name of the person who did the review is available, and also allows multiple people to review a single patch if they wish. There's no harm in multiple reviews: in fact, with several reviews the patch is more likely to get committed with speed.

If you find a problem in the code, note that in a comment on the bug while removing the patch keyword. Removing the patch keyword will take the patch out of the "patches waiting" list, and then the assignee of the bug can add it back again when a fixed patch is ready. Please be as specific as possible when noting problems with patches — treat it like a bug report and give all relevant information that the assignee might need to work out what you are talking about.

If you have access to a private LJ installation, you can go one step further and actually test patches. They still must be reviewed before commiting to CVS, but also testing is good for larger patches. You can mark individual patch attachments as 'tested' and note in the bug that you have tested them.

There are lots of patches sitting in Zilla waiting to be committed. Please help review!

link2 comments|post comment

Input needed [Sep. 7th, 2002|03:31 pm]
LiveJournal Development

lj_dev

[firelegend]
portal box's in re-design
link5 comments|post comment

(no subject) [Sep. 7th, 2002|04:10 pm]
LiveJournal Development

lj_dev

[firelegend]
whats this do (from index.bml):

?_code

return '' if (BML::get_client_header('User-Agent') =~ /AOL \d/);
return "p align='center' style='font-size: 1.7em'a href='/create.bml'?_ml .boldcreate _ml?/a/p";

_code?
link6 comments|post comment

intro [Sep. 7th, 2002|08:04 pm]
LiveJournal Development
lj_dev
[amyvartdal]
[music |Dire Straits - Down to the waterline]

I think I saw something about posting an intro, so here I go.

Hi. I'm Amy. I'm installing LJ on my machine at the moment(Just need to beat Apache and mod_perl). What I want to do is make it possible to retrivie number of comments made to a post(I've seen earlier post about it) and make my own diary interface with LJ so that it'll upload new comments to my LJ. So... Hi :)
link2 comments|post comment

navigation
[ viewing | September 7th, 2002 ]
[ go | Previous Day|Next Day ]