Closed Bug 739072 Opened 13 years ago Closed 13 years ago

Fix "thoes" spelling

Categories

(Release Engineering :: General, defect, P4)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
VERIFIED FIXED

People

(Reporter: sgautherie, Assigned: nickyekaiqi)

References

(
URL
)

Details

Attachments

(2 files, 2 obsolete files)

Modified patch following the instruction
(deleted), patch
coop
: review+
Details | Diff | Splinter Review
modify r to f
(deleted), patch
Details | Diff | Splinter Review
Use "those". "Found one matching line"
I am interested in this bug. Seems a little trivial :) But it is a good first bug to learn how to use mxr and creating a patch. So could someone kindly assign it to me?
(In reply to Ye Kaiqi from comment #1) > could someone kindly assign it to me? You may be able to do it when attaching your patch, otherwise I'll help.
Attached patch Do as instruction [Whole file] (obsolete) (deleted) — — Splinter Review
Hope what I did is correct for creating a patch!
I created the patch wrongly for the last patch. I uploaded a new one
Attached patch spelling change patch (obsolete) (deleted) — — Splinter Review
Attachment #610397 - Attachment is obsolete: true
Attachment #610397 - Attachment description: Do as instruction → Do as instruction [Whole file]
Assignee: nobody → nickyekaiqi
Status: NEW → ASSIGNED
Attachment #610437 - Attachment is obsolete: true
Attachment #610437 - Flags: review?(coop)
Attachment #610479 - Flags: review?
Comment on attachment 610479 [details] [diff] [review] Modified patch following the instruction >Bug 739072 : Change spelling error "thoes" to "those"; r=sgautherie f=sgautherie
Attachment #610479 - Flags: review? → review?(coop)
Attachment #610479 - Flags: review?(coop) → review+
So it has been reviewed? I am quite new to it and want to start other bugs after this one
Anything else to do with this bug?
(In reply to Ye Kaiqi from comment #9) > So it has been reviewed? Yes. (In reply to Ye Kaiqi from comment #10) > Anything else to do with this bug? Fix comment 8.
Attached patch modify r to f (deleted) — — Splinter Review
Keywords: checkin-needed
Whiteboard: [good first bug][mentor=sgautherie][lang=py]
https://hg.mozilla.org/integration/mozilla-inbound/rev/6990e7e757c3 Thanks for the patch! Feel free to drop in to #developers on irc.mozilla.org if you ever want help finding other bugs to work on.
Keywords: checkin-needed
(In reply to Ryan VanderMeulen from comment #13) > https://hg.mozilla.org/integration/mozilla-inbound/rev/6990e7e757c3 > > Thanks for the patch! Feel free to drop in to #developers on irc.mozilla.org > if you ever want help finding other bugs to work on. I was in #developers for several days for the information of another bug! Thanks for your help!
https://hg.mozilla.org/mozilla-central/rev/6990e7e757c3
Status: ASSIGNED → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
V.Fixed, per MXR search.
Status: RESOLVED → VERIFIED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: