Closed Bug 1647254 Opened 1 year ago Closed 1 year ago

parse_scalars.py: Undefined variable 'MAX_LENGTH_COUNT' E0602 (pylint)

Categories

(Toolkit :: Telemetry, defect)

defect

Tracking

()

RESOLVED FIXED
mozilla79
Tracking Status
firefox79 --- fixed

People

(Reporter: Sylvestre, Assigned: ash153311, Mentored)

References

(Blocks 1 open bug)

Details

(Keywords: good-first-bug, Whiteboard: [lang=python])

Attachments

(1 file)

/home/sylvestre/dev/mozilla/mozilla-central.hg/toolkit/components/telemetry/build_scripts/mozparsers/parse_scalars.py
  198:79  error  Undefined variable 'MAX_LENGTH_COUNT'  E0602 (pylint)

The error message won't be displayed

To reproduce :

$  ./mach lint -l pylint -n toolkit/components/telemetry/build_scripts/mozparsers/parse_scalars.py   

As the change is trivial, it is just to learn how to contribute to Firefox.

Tutorial to contribute:
https://firefox-source-docs.mozilla.org/contributing/contribution_quickref.html

Please don't ask for the bug to be assigned. It will be automatically assigned to the first patch.

Summary: Undefined variable 'MAX_LENGTH_COUNT' E0602 (pylint) → parse_scalars.py: Undefined variable 'MAX_LENGTH_COUNT' E0602 (pylint)
No longer blocks: 1623024
Blocks: pylint
Assignee: nobody → ash153311
Status: NEW → ASSIGNED
Pushed by abutkovits@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/5b9c0602e19d
Change 'MAX_LENGTH_COUNT' to 'MAX_KEY_LENGTH' r=sylvestre
Status: ASSIGNED → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla79

Taegeon, if you want, you can add toolkit/components/telemetry/ in the include list of tools/lint/pylint.yml, thanks

(In reply to Sylvestre Ledru [:Sylvestre] from comment #4)

Taegeon, if you want, you can add toolkit/components/telemetry/ in the include list of tools/lint/pylint.yml, thanks

Can I create a good-first-bug for other new contributors?

Flags: needinfo?(sledru)

sounds great, thanks :)

Flags: needinfo?(sledru)

I completed to create bug 1648027.

See Also: → 1648027
You need to log in before you can comment on or make changes to this bug.