Featured Badge Names Type

[bryantkelley]

Description

Removes the featuredBadgeNames string array and replaces it with a Type for narrowing and improved tree-shaking.

Fixes # (issue)

Type of Change

• Bug fix (non-breaking change which fixes an issue)
• New feature (non-breaking change which adds functionality)
• Breaking change (fix or feature that would cause existing functionality to not work as expected)

How Has This Been Tested?

I linked these changes to victory with pnpm and do not see unused SVGs when building the victory docs site.

Checklist: (Feel free to delete this section upon completion)

• I have included a changeset if this change will require a version change to one of the packages.
• I have performed a self-review of my own code
• I have commented my code, particularly in hard-to-understand areas
• I have made corresponding changes to the documentation
• I have run all builds, tests, and linting and all checks pass
• I have added tests that prove my fix is effective or that my feature works
• My changes generate no new warnings
• Any dependent changes have been merged and published in downstream modules

[changeset-bot]

🦋 Changeset detected

Latest commit: 4013520

The changes in this PR will be included in the next version bump.

This PR includes changesets to release 1 package

Name	Type
formidable-oss-badges	Minor

Not sure what this means? Click here to learn what changesets are.

Click here if you're a maintainer who wants to add another changeset to this PR

[JDMathew]

I'm bumping into this change when updating React Native AMA's docs. While this change does improve tree-shaking it might be useful to have a definite list of featured projects (featuredLogoNames) that is exported from formidable-oss-badges while still maintaining the tree-shaking this change implements. I'm happy to put a PR together for this.