Closed Bug 1687947 Opened 4 years ago Closed 4 years ago

Atom declarations formatted across multiple lines

Categories

(Developer Infrastructure :: Lint and Formatting, defect)

Product:
Developer Infrastructure
Component:
Lint and Formatting
Type:
defect

Tracking

(firefox-esr78 unaffected, firefox84 wontfix, firefox85 wontfix, firefox86 fixed)

Status:
RESOLVED FIXED
Milestone:
86 Branch
Tracking Flags:
Tracking Status
firefox-esr78 --- unaffected
firefox84 --- wontfix
firefox85 --- wontfix
firefox86 --- fixed

People

(Reporter: hsivonen, Assigned: sheehan)

References

(Regression)

Details

(Keywords: regression)

Attachments

(1 file)

Bug 1687947: keep `Atom` declarations on a single line and exclude from `black` formatting r?hsivonen
(deleted), text/x-phabricator-request
Details

StaticAtoms.py has this comment:

# Please keep the Atom() definitions on one line as this is parsed by the
#   htmlparser: parser/html/java/htmlparser

Yet, there are now multiline atom declarations.

Please exclude the atom declarations from Python formatting so that each declaration can be on one line.

Has Regression Range: --- → yes

These lines are parsed by the htmlparser and are expected to be on a single
line. The black reformat has moved some of these definitions to multiple lines
due to line length. This commit moves all declarations back to a single line and
adds fmt: {off,on} statements so they will be ignored in future reformats.

Assignee: nobody → sheehan
Status: NEW → ASSIGNED
Pushed by cosheehan@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/8abc47cc7404 keep `Atom` declarations on a single line and exclude from `black` formatting r=hsivonen

https://hg.mozilla.org/mozilla-central/rev/8abc47cc7404

Status: ASSIGNED → RESOLVED
Closed: 4 years ago
status-firefox86: --- → fixed
Resolution: --- → FIXED
Target Milestone: --- → 86 Branch

Set release status flags based on info from the regressing bug 1654103

status-firefox84: --- → affected
status-firefox85: --- → affected
status-firefox-esr78: --- → unaffected
Product: Firefox Build System → Developer Infrastructure
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: