Skip to content

chore: remove non-standard _EXTRA instances#21365

Merged
david-allison merged 1 commit into
ankidroid:mainfrom
ericli3690:ericli3690-extra-naming-standardization
Jul 12, 2026
Merged

chore: remove non-standard _EXTRA instances#21365
david-allison merged 1 commit into
ankidroid:mainfrom
ericli3690:ericli3690-extra-naming-standardization

Conversation

@ericli3690

Copy link
Copy Markdown
Member

Note

Assisted-by: Composer 2.5

Purpose / Description

While working on a PR for fixing up naming conventions, I found some extras that have EXTRA rather than a leading EXTRA in their name. It was annoying me, so here's a quick commit to fix it. I originally tried fully enforcing the leading EXTRA_ convention across all intent extras, but that got out of control really quickly, so here's a smaller incremental PR for now.

Fixes

Approach

Simple renaming.

How Has This Been Tested?

App builds, unit tests pass.

Checklist

  • You have a descriptive commit message with a short title (first line, max 50 chars).
  • You have commented your code, particularly in hard-to-understand areas
  • You have performed a self-review of your own code

While working on a PR for fixing up naming conventions, I found some extras that have _EXTRA rather than a leading EXTRA_ in their name. It was annoying me, so here's a quick commit to fix it. I originally tried fully enforcing the leading EXTRA_ convention across all intent extras, but that got out of control really quickly, so here's a smaller incremental PR for now.

Assisted-by: Composer 2.5
@ericli3690 ericli3690 self-assigned this Jul 12, 2026
@david-allison
david-allison added this pull request to the merge queue Jul 12, 2026
Merged via the queue into ankidroid:main with commit 69eeb88 Jul 12, 2026
20 checks passed
@github-actions github-actions Bot added this to the 2.25 release milestone Jul 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants