Skip to content

Extended-tokens: Cleanup token exports - #34384

Merged
Mitch-At-Work merged 4 commits into
microsoft:extended-tokensfrom
Mitch-At-Work:user/mifraser/unset-backup-tokens
May 8, 2025
Merged

Extended-tokens: Cleanup token exports#34384
Mitch-At-Work merged 4 commits into
microsoft:extended-tokensfrom
Mitch-At-Work:user/mifraser/unset-backup-tokens

Conversation

@Mitch-At-Work

Copy link
Copy Markdown
Contributor

New Behavior

  • Removed figma only and null tokens from script export
  • Enabled 'unset' variables to be fully null even when a fluent fallback is provided

@Mitch-At-Work
Mitch-At-Work marked this pull request as ready for review May 6, 2025 23:22
@Mitch-At-Work
Mitch-At-Work requested a review from a team as a code owner May 6, 2025 23:22
@github-actions

github-actions Bot commented May 6, 2025

Copy link
Copy Markdown

Pull request demo site: URL

Comment thread packages/semantic-tokens/scripts/tokenGen.ts Outdated

@brandonthomas brandonthomas left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Can we consolidate the valid token check or make it more comprehensive?

@Mitch-At-Work
Mitch-At-Work merged commit 6dd3bec into microsoft:extended-tokens May 8, 2025
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