feat(Tag)!: remove deprecated props and update API

[booc0mtaco]

Summary:

• remove deprecated value "yield" from variant list

We had both "warning" and "yield" as possible values for this component's variants, which was confusing. Going forward, we will start with just "warning", and remove this option from the component list. If using "yield", please consider using "warning" instead, or some other treatment.

• change "icon" prop to be of type IconName to be consistent with all other components

This syncs up the headless Icon handling across all components, meaning that the names and possible values are determined by the mapped icons, and not any possible ReactNode instance. For consumers, replace any usages of <Icon /> with the string provided to this cmoponent. Any other nodes used will have to be removed.

• tidy up usage of <Text /> and typography handling

simplify the handling of type in this component by moving to use preset instead of the more <font> like weight and size props, which will also be removed.

• update tests, snapshots, and usages.

Test Plan:

• update existing stories and tests
• Wrote automated tests
• CI tests / new tests are not applicable
• Manually tested my changes, but I want to keep the details secret
• Manually tested my changes, and here are the details:
  • Create an alpha publish and try out in edu-stack or traject as a sanity check if changes affect build or deploy, or are breaking, such as token changes, widely used component updates, hooks changes, and major dependency upgrades.

[codecov]

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 96.65%. Comparing base (9ce8625) to head (e08d2eb).

Additional details and impacted files

@@           Coverage Diff           @@
##              v14    #1862   +/-   ##
=======================================
  Coverage   96.65%   96.65%           
=======================================
  Files         108      108           
  Lines        2151     2153    +2     
  Branches      574      575    +1     
=======================================
+ Hits         2079     2081    +2     
  Misses         70       70           
  Partials        2        2           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[ahuth]

LGTM

[booc0mtaco]

@jeremiah-clothier will try using size or distinct_size (referenced in the doc.s) to detect the number of commits to see how far back to reach for commitlint. Wondering if you tried either of those already, and they had some issue?

[jeremiah-clothier]

@jeremiah-clothier will try using size or distinct_size (referenced in the doc.s) to detect the number of commits to see how far back to reach for commitlint. Wondering if you tried either of those already, and they had some issue?

Ohh maybe, I can look at this Thursday again. Rn the only workaround is to just not use any ' characters in commits. I'm pretty sure that's the issue

[booc0mtaco]

@jeremiah-clothier will try using size or distinct_size (referenced in the doc.s) to detect the number of commits to see how far back to reach for commitlint. Wondering if you tried either of those already, and they had some issue?

Ohh maybe, I can look at this Thursday again. Rn the only workaround is to just not use any ' characters in commits. I'm pretty sure that's the issue

Yep, confirmed on two commits (one working and one not working). I'll try that out separate from this. if we merge a nice solution in next i can merge that so that it gets over here for this change