1554884 - Simplify devtools/client/netmonitor/src/utils/filter-text-utils.js and remove eslint-disable complexity

Status: RESOLVED FIXED

(DevTools :: Netmonitor, task, P3)

Description

[Patrick Brosset <:pbro>]

This file contains a special eslint comment to temporarily disable the complexity eslint rule since it contains one or more functions that exceed the maximum complexity threshold. The code should be simplified, and the comment removed. Link to the code in question: https://searchfox.org/mozilla-central/search?q=eslint-disable+complexity&case=false&regexp=false&path=devtools%2Fclient%2Fnetmonitor%2Fsrc%2Futils%2Ffilter-text-utils.js

Comment 1

[Arun Kumar Mohan]

Attachment: Bug 1554884 - Refactor isFlagFilterMatch function to remove disabled complexity eslint rule. r=pbro

Comment 2

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Just NI myself

Honza

Comment 3

[Jan Honza Odvarko [:Honza] (always need-info? me)]

Review comments posted
Honza

Comment 4

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/integration/autoland/rev/12374bb26e7b
Refactor isFlagFilterMatch function to remove disabled complexity eslint rule. r=Honza

Comment 5

[Daniel Varga [:dvarga]]

https://hg.mozilla.org/mozilla-central/rev/12374bb26e7b