Fix a ruff warning in js/src/jit/GenerateABIFunctionType.py
Categories
(Developer Infrastructure :: Lint and Formatting, task)
Tracking
(firefox143 fixed)
| Tracking | Status | |
|---|---|---|
| firefox143 | --- | fixed |
People
(Reporter: Sylvestre, Assigned: viet-hung.nhu)
References
(Blocks 1 open bug)
Details
(Keywords: good-first-bug, Whiteboard: [lang=python])
Attachments
(1 external file)
Filling as a good first bug to learn workflows.
js/src/jit/GenerateABIFunctionType.py
72:12: warning Consider merging multiple comparisons: `arg in {"Float32", "Float64"}`. PLR1714 (ruff)
Link to the code:
https://searchfox.org/mozilla-central/source/js/src/jit/GenerateABIFunctionType.py#72
To run the linter:
./mach lint -W -l ruff js/src/jit/GenerateABIFunctionType.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
https://firefox-source-docs.mozilla.org/contributing/stack_quickref.html
Please don't ask for the bug to be assigned. It will be automatically assigned to the first patch.
| Assignee | ||
Comment 1•1 year ago
|
||
Updated•1 year ago
|
Comment 2•1 year ago
|
||
There is an r+ patch which didn't land and no activity in this bug for 2 weeks.
:viet-hung.nhu, could you have a look please?
If you still have some work to do, you can add an action "Plan Changes" in Phabricator.
For more information, please visit BugBot documentation.
| Reporter | ||
Updated•1 year ago
|
Comment 4•1 year ago
|
||
| bugherder | ||
Description
•