If you really have to fix a bug right away, file it and send a notice to cldr-dev explaining what you are doing and why. Otherwise, file the bug, wait for the next Wednesday meeting for discussion and approval.
- You can add set other fields, but don't change the Owner. That way the committee sees the incoming tickets (those marked as Owner: somebody).
- The committee assesses the tickets weekly.
- Straightforward ones are just assigned and prioritized and given a milestone.
- Questionable ones are assigned, but left with priority=assess. The assignee will report back before doing anything.
- More complicated ones are moved into the
design folder for later discussion at the meeting.
- Don't fix a bug until it's been approved.
- If a ticket has multiple owners, the Notes must describe who is doing what. Otherwise it needs to be split into multiple tickets.
- Assignees periodically review and take care of their tickets.
- Go to http://unicode.org/cldr/trac/report/14 for My Tickets
- assess: Tickets marked "assess" need to be evaluated. Send your suggested disposition to the cldr group by email.
- duplicate: If the ticket is a duplicate, any extra information is copied to the other ticket, an xref is set in both, and the ticket is returned.
- If the ticket requires separate phases, such as a change to the spec, then a separate ticket is filed, with xrefs set in both.
- If part of a ticket cannot be completed in the release:
- Set the milestone to the release number
- Add a comment indicating what part of the ticket was done, and what part wasn't.
- Set a reviewer.
- Create a new ticket for the remaining work, with an xref to the old bug.
- If the ticket can't be done due to lack of time, or is being downgraded in priority from major or above to below major, send an email to the cldr group.
- If during the course of doing the ticket, the person runs up
against a problem, then send a message to the cldr group. Examples:
- The description doesn't work
- Another mechanism looks better
Survey Tool in Production Phase!
- Before you make a commit, add a line to the Issues spreadsheet below (Spreadsheet View), with status OPEN, bug number, short desc.
When you're ready for checking, add the commit number to your line, send a message to a designee (cc cldr-dev) and change the status to CHECK, add testing description (what to check, links if possible).Designee changes status to READY, or communicates problems back to you.
- Don't do this if there is a status=PUSHING
Once the statuses are all READY, anyone can push to production:
- All changes checked into SVN for the ticket use the format "cldrbug XXX: <description>", where XXX is the ticket number.
- Don't mark the bug as fixed! Instead,
- Once the ticket is done, assign a reviewer.
- Send a message to the reviewer if the review should be done soon.
- On "Available Reports", click on My Reviews. Open each of your tickets.
- In the top right, under View Tickets, you'll see "Review <x> commits."
- Click on "Review" to see the changed files. Follow the instructions at the bottom.
- The reviewer needs to look at the Notes and Replies in the original ticket, since
they may contain changes or additions, and also sometimes alert the
reviewer to changes not captured by the diff tool.
- The reviewer looks through the diff to make sure that the
implementation matches the description. If any problems are found, the
reviewer contacts the assignee, and they work together until the
problems are resolved.
- For data tickets, the goal is to verify that the data in the ticket matches what is entered in.
- If any issues come up or the topic needs review, send a message to the firstname.lastname@example.org list.
- If the implementation looks good, resolve as "fixed".
- Change PUSHING to PUSHED
- All reviewers should be done well before the end of the release, so that any problems can be taken care of.
- At release time, all the tickets with status "fixed" are changed to "closed".
Start of Release
- The future folder tickets are moved to the discuss folder
- Unscheduled tickets (with no release number) are re-evaluated.