Closed Bug 2015841 Opened 9 months ago Closed 7 months ago

Update to ruff 0.15.0

Categories

(Developer Infrastructure :: Lint and Formatting, task)

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

Tracking

(firefox151 fixed)

Status:
RESOLVED FIXED
Milestone:
151 Branch
Tracking Flags:
Tracking Status
firefox151 --- fixed

People

(Reporter: Sylvestre, Assigned: Sylvestre)

References

(Blocks 1 open bug)

Details

Attachments

(4 external files)

only one change

Pushed by sledru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/7afa00194d15 https://hg.mozilla.org/integration/autoland/rev/69131d67d71c ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,ahal,sparky
Pushed by nfay@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/b062ded6d83e https://hg.mozilla.org/integration/autoland/rev/072e7983a7da Revert "Bug 2015841 - ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,ahal,sparky" for causing multiple bustages

Backed out for causing multiple bustages

Backout link

Push with failures

Flags: needinfo?(sledru)
Flags: needinfo?(sledru)
Attachment #9543922 - Attachment description: Bug 2015841 - ruff/format: reformat the tree with 0.15.0 r?#linter-reviewers → Bug 2015841 - Reformat the code with ruff-format 0.15.0 r?#linter-reviewers
Pushed by sledru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/cc04e8271b65 https://hg.mozilla.org/integration/autoland/rev/c1cea4d53f42 Update to ruff 0.15.0 r=linter-reviewers,ahal https://github.com/mozilla-firefox/firefox/commit/93b38f0d7972 https://hg.mozilla.org/integration/autoland/rev/a3f0c24b1768 Reformat the code with ruff-format 0.15.0 r=linter-reviewers,perftest-reviewers,mozperftest-reviewers,releng-reviewers,sessionstore-reviewers,dom-storage-reviewers,dom-worker-reviewers,whimboo,chutten,ahal,sparky,sclements https://github.com/mozilla-firefox/firefox/commit/654a09320c16 https://hg.mozilla.org/integration/autoland/rev/f38fa8b9ae58 ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille https://github.com/mozilla-firefox/firefox/commit/5eca473e96a8 https://hg.mozilla.org/integration/autoland/rev/92de1a951845 ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements
Pushed by imoraru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/c616d6af1dda https://hg.mozilla.org/integration/autoland/rev/a435070fd6c2 Revert "Bug 2015841 - ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements" for causing multiple android reftest failures.

Revert for causing multiple android reftest failures.

Flags: needinfo?(sledru)

i can't reproduce:
https://treeherder.mozilla.org/jobs?repo=try&revision=5522b0e66d13acc8ca578437793528584f904a35
and i don't see why it would cause this issue.

Trying to land it again

Flags: needinfo?(sledru)
Pushed by sledru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/4b0785d39ed1 https://hg.mozilla.org/integration/autoland/rev/cf7ca6ffbefb Update to ruff 0.15.0 r=linter-reviewers,ahal https://github.com/mozilla-firefox/firefox/commit/15a20a737e57 https://hg.mozilla.org/integration/autoland/rev/ae28a96ebac4 Reformat the code with ruff-format 0.15.0 r=linter-reviewers,perftest-reviewers,mozperftest-reviewers,releng-reviewers,sessionstore-reviewers,dom-storage-reviewers,dom-worker-reviewers,whimboo,chutten,ahal,sparky,sclements https://github.com/mozilla-firefox/firefox/commit/1e0b4c3c7b41 https://hg.mozilla.org/integration/autoland/rev/7b17713b525b ruff: fix 'String is split more times than necessary' r=linter-reviewers,ahal,sergesanspaille https://github.com/mozilla-firefox/firefox/commit/0382300c2d1c https://hg.mozilla.org/integration/autoland/rev/37a6ff8b3253 ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements
Pushed by chorotan@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/f807c9b27e79 https://hg.mozilla.org/integration/autoland/rev/c87ae28dce51 Revert "Bug 2015841 - ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,ahal,sparky,sclements" for causing lint failure

Backed out for causing multiple failures

Backout link

Push with failures

Failure log
Failure log
Failure log
Failure log

Flags: needinfo?(sledru)

ok, thanks :)

Flags: needinfo?(sledru)
Pushed by sledru@mozilla.com: https://github.com/mozilla-firefox/firefox/commit/87a13e0ed000 https://hg.mozilla.org/integration/autoland/rev/cbc04bab9629 Update to ruff 0.15.0 r=linter-reviewers,ahal https://github.com/mozilla-firefox/firefox/commit/4e1cd03c9590 https://hg.mozilla.org/integration/autoland/rev/503d84194147 Reformat the code with ruff-format 0.15.0 r=linter-reviewers,perftest-reviewers,mozperftest-reviewers,releng-reviewers,sessionstore-reviewers,dom-storage-reviewers,dom-worker-reviewers,whimboo,chutten,ahal,sparky,sclements https://github.com/mozilla-firefox/firefox/commit/bdd074022bd8 https://hg.mozilla.org/integration/autoland/rev/31b20a902221 ruff: fix 'String is split more times than necessary' r=linter-reviewers,perftest-reviewers,ahal,sergesanspaille,sparky https://github.com/mozilla-firefox/firefox/commit/8af186a748f6 https://hg.mozilla.org/integration/autoland/rev/1079b20cd7fe ruff: fix 'Lambda may be unnecessary; consider inlining inner function' r=linter-reviewers,perftest-reviewers,sessionstore-reviewers,chutten,ahal,sparky,sclements
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: