Closed Bug 1874861 Opened 2 years ago Closed 2 years ago

Remove redundant LongParameterList suppressions based on the updated Detekt config

Categories

(Firefox for Android :: Tooling, task, P1)

All
Android
task

Tracking

()

RESOLVED FIXED
123 Branch
Tracking Status
firefox123 --- fixed

People

(Reporter: towhite, Assigned: towhite)

References

(Blocks 1 open bug)

Details

Attachments

(1 file)

59 bytes, text/x-github-pull-request
Details | Review

Remove all redundant LongParameterList suppressions.

LongParameterList has been updated to allows for 8 params - this was previously 6.

Status: NEW → ASSIGNED
Depends on: detekt/ktlint/lint
Priority: P3 → P1

The Android team has not been keeping our P1 bug list up to date, so we're resetting all our P1 bugs to P2 to avoid signalling that we're actively working on bugs that we're not. The BMO documentation https://wiki.mozilla.org/BMO/UserGuide/BugFields#priority says P1 means "fix in the current release cycle" and P2 means "fix in the next release cycle or the following (nightly + 1 or nightly + 2)".

If you are actively working on this bug and expect to ship it in Fx 122 or 123, then please restore the priority back to P1.

Priority: P1 → P2
Priority: P2 → P1
Attached file GitHub Pull Request
Status: ASSIGNED → RESOLVED
Closed: 2 years ago
Resolution: --- → FIXED
Target Milestone: --- → 123 Branch
No longer depends on: detekt/ktlint/lint
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: