Skip to content

Removed deleted files from themes.config.ts #1214

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
merged 2 commits into from
May 6, 2025

Conversation

lukasoppermann
Copy link
Contributor

Summary

List of notable changes:

What should reviewers focus on?

Steps to test:

@Copilot Copilot AI review requested due to automatic review settings May 5, 2025 19:07
@lukasoppermann lukasoppermann requested review from a team as code owners May 5, 2025 19:07
@lukasoppermann lukasoppermann requested a review from langermank May 5, 2025 19:07
Copy link

changeset-bot bot commented May 5, 2025

🦋 Changeset detected

Latest commit: 7e2b6c2

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

This PR includes changesets to release 1 package
Name Type
@primer/primitives Patch

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

Copy link
Contributor

@Copilot Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull Request Overview

This PR cleans up scripts/themes.config.ts by removing references to deleted token files and adjusting the order of the component token source entry in each theme definition.

  • Removed source entries for deleted color/light/*, color/dark/*, and related override files
  • Reordered src/tokens/component/*.json5 to appear after color/*.json5 in every theme's source array

@lukasoppermann lukasoppermann merged commit f002fca into main May 6, 2025
27 checks passed
@lukasoppermann lukasoppermann deleted the rm-old-files-from-themesConfig branch May 6, 2025 00:02
@primer primer bot mentioned this pull request May 6, 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