Skip to content

Commit

Permalink
For mozilla-mobile#21294: suppress MaxLineLength in Fact.toEvent.
Browse files Browse the repository at this point in the history
These double comparisons are easier to read and see the pattern of on one line
so I'd rather keep them on one line. Additionally, it's difficult to
test each change individually so I'd rather not make additional changes.
To do this, I suppressed the max line length warning.
  • Loading branch information
mcomella authored and mergify[bot] committed Sep 30, 2021
1 parent c10f411 commit 365983d
Showing 1 changed file with 1 addition and 1 deletion.
Original file line number Diff line number Diff line change
Expand Up @@ -157,7 +157,7 @@ internal class ReleaseMetricController(
MetricServiceType.Marketing -> isMarketingDataTelemetryEnabled()
}

@Suppress("LongMethod")
@Suppress("LongMethod", "MaxLineLength")
private fun Fact.toEvent(): Event? = when {
Component.FEATURE_PROMPTS == component && LoginDialogFacts.Items.DISPLAY == item -> Event.LoginDialogPromptDisplayed
Component.FEATURE_PROMPTS == component && LoginDialogFacts.Items.CANCEL == item -> Event.LoginDialogPromptCancelled
Expand Down

0 comments on commit 365983d

Please sign in to comment.