1564313 - Use the // common style in the nursery code.

Status: RESOLVED FIXED

(Core :: JavaScript: GC, task, P3)

Comment 1

[Paul Bone [:pbone]]

Hi sfink,

You recently said that you found a way to add something to the summary line of your patches when they don't change semantics so that searchfox.org will skip past them when annotating code? Could you remind me how to do that?

Comment 2

[Paul Bone [:pbone]]

Attachment: Bug 1564313 - Update comments to // style in nursery code r=sfink

Comment 3

[Paul Bone [:pbone]]

Attachment: Bug 1564313 - Make some style changes to prose in comments r=sfink

Depends on D37365

Comment 4

[Steve Fink [:sfink] [:s:]]

(In reply to Paul Bone [:pbone] from comment #1)

Hi sfink,

You recently said that you found a way to add something to the summary line of your patches when they don't change semantics so that searchfox.org will skip past them when annotating code? Could you remind me how to do that?

Yeah, and that was harder to dig up than it should be. It looks like, according to https://bugzilla.mozilla.org/show_bug.cgi?id=1508002 , that you can use the magic string "ignore-this-changeset" in a commit message.

Comment 5

[Paul Bone [:pbone]]

Thanks.

Should we document that somewhere? Where did you search first because that could be the right place to put it.

Comment 6

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/a5c589bdfa0d
Update comments to // style in nursery code r=sfink
https://hg.mozilla.org/integration/autoland/rev/4ac25fd8120a
Make some style changes to prose in comments r=sfink

Comment 7

[Andreea Pavel [:apavel]]

https://hg.mozilla.org/mozilla-central/rev/a5c589bdfa0d
https://hg.mozilla.org/mozilla-central/rev/4ac25fd8120a