1362510 - Remove services/sync/locales/en-US/errors.properties

Status: RESOLVED FIXED

(Firefox :: Sync, enhancement)

Description

[Edouard Oger [:eoger]]

I thought I removed this but it turns out I didn't. It is not used anymore.

Comment 1

[nbeltran14]

I think I might be interested in this bug as it seems simple enough for someone who has not had any experience fixing them. First, just so I understand, is this removing the entire errors.properties file from en-US?

Comment 2

[Edouard Oger [:eoger]]

Hello nbeltran14,

Yes you are correct. We also need to remove the references to that file:
- browser/base/content/test/static/browser_all_files_referenced.js contains a reference.
- services/sync/locales/jar.mn also contains another.

Thanks for picking this up!

Comment 3

[nbeltran14]

I'm planning to push a patch, but before I do that I'm going to create a new patch holder using Mercurial's hg qnew command. Is it preferable to state in the message that the mentor of the bug be the one to review the patch? I'm still very new to this.

Comment 4

[Edouard Oger [:eoger]]

Yes, for simplicity I will be the one to review your patch.
Your commit message should follow the following format:

Bug <Bug number> - <What your commit accomplishes (not the way you do it!)>. r?<reviewer(s)>

For example, for this bug your commit message should be something like:

Bug 1362510 - Remove errors.properties and references. r?eoger

Comment 5

[jomer14]

Hi I am a team mate of nbeltran14 in class. We are working together on this bug and I was just letting you know that you may receive a patch from me.

Comment 6

[nbeltran14]

(In reply to jomer14 from comment #5)
> Hi I am a team mate of nbeltran14 in class. We are working together on this
> bug and I was just letting you know that you may receive a patch from me.

I can confirm that jomer14 is a class mate of mine and that we are working together on things.

Comment 7

[jomer14]

Attachment: Bug 1362510 - 'Remove errors.properties and references.'

Review commit: https://reviewboard.mozilla.org/r/138512/diff/#index_header
See other reviews: https://reviewboard.mozilla.org/r/138512/

Comment 8

[Edouard Oger [:eoger]]

Comment on attachment 8866894 [details]
Bug 1362510 - 'Remove errors.properties and references.'

https://reviewboard.mozilla.org/r/138512/#review141726

Almost there, change the commit message and we're good to go. Thanks!

::: commit-message-b9bd7:1
(Diff revision 1)
> +bug1362510 - 'Removed errors.properties and references because the file is no longer in use.' r?eoger

Please follow the model I described in comment 4

Comment 9

[jomer14]

So do you want it exactly the way you have it typed in comment 4? As in "Bug 1362510 - Remove errors.properties and references. r?eoger" Thanks!

Comment 10

[Edouard Oger [:eoger]]

As long as you respect the format [0] and your commit message makes sense I'll accept it.
Check-out the other commit messages with |hg log -l 30| if you need more examples.


[0] Bug <Bug number> - <What your commit accomplishes (not the way you do it!)>. r?<reviewer(s)>

Comment 11

[jomer14]

Comment on attachment 8866894 [details]
Bug 1362510 - 'Remove errors.properties and references.'

Review request updated; see interdiff: https://reviewboard.mozilla.org/r/138512/diff/1-2/

Comment 12

[Edouard Oger [:eoger]]

Comment on attachment 8866894 [details]
Bug 1362510 - 'Remove errors.properties and references.'

https://reviewboard.mozilla.org/r/138512/#review142582

Comment 13

[Edouard Oger [:eoger]]

Thanks!
https://hg.mozilla.org/integration/mozilla-inbound/rev/20c570b957c6

Comment 14

[Wes Kocher (:KWierso) (Not reading bugmail; email directly if needed)]

https://hg.mozilla.org/mozilla-central/rev/20c570b957c6