Closed Bug 1930706 Opened 1 year ago Closed 1 year ago

Clean up lint-baseline.xml unused resources

Categories

(Firefox for Android :: Onboarding, task)

All
Android
task

Tracking

()

RESOLVED FIXED
135 Branch
Tracking Status
firefox135 --- fixed

People

(Reporter: Gela, Assigned: Gela)

References

Details

(Whiteboard: [fxdroid])

Attachments

(1 file)

To avoid the linter failing for now, some resources are marked as unused to ignore warnings but once those are in use we need to clean up and remove this bit:

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_pick_a_theme appears to be unused">
        <location file="src/main/res/drawable/ic_pick_a_theme.xml" />
    </issue>

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_pick_a_theme_dark appears to be unused">
        <location file="src/main/res/drawable/ic_pick_a_theme_dark.xml" />
    </issue>

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_pick_a_theme_light appears to be unused">
        <location file="src/main/res/drawable/ic_pick_a_theme_light.xml" />
    </issue>

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_pick_a_theme_system_auto appears to be unused">
        <location file="src/main/res/drawable/ic_pick_a_theme_system_auto.xml" />
    </issue>

Also this bit:

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_onboarding_customize_toolbar appears to be unused">
        <location file="src/main/res/drawable/ic_onboarding_customize_toolbar.xml" />
    </issue>

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_onboarding_bottom_toolbar appears to be unused">
        <location file="src/main/res/drawable/ic_onboarding_bottom_toolbar.xml" />
    </issue>

    <issue id="UnusedResources"
        message="The resource R.drawable.ic_onboarding_top_toolbar appears to be unused">
        <location file="src/main/res/drawable/ic_onboarding_top_toolbar.xml" />
    </issue>
Whiteboard: [fxdroid]

Since the assets related to theme selection in onboarding are now in use, we
can safely remove the linter overrides

Pushed by gmalekpour@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/6a485bec198a Clean up lint-baseline.xml unused resources r=android-reviewers,twhite,tjorjani
Status: NEW → RESOLVED
Closed: 1 year ago
Resolution: --- → FIXED
Target Milestone: --- → 135 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: