Skip to content

refactor: merge m3 mdc and mat tokens #30755

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 4 commits into from
Apr 1, 2025

Conversation

andrewseguin
Copy link
Contributor

@andrewseguin andrewseguin commented Mar 31, 2025

Combines token files if they match the same prefix. Attempts to normalize the files to match the same format and improve readability. Removes many unused variables that were provided by MDC but not used by our components

@andrewseguin andrewseguin added the target: major This PR is targeted for the next major release label Mar 31, 2025
@andrewseguin andrewseguin requested a review from a team as a code owner March 31, 2025 21:58
@andrewseguin andrewseguin requested review from crisbeto and mmalerba and removed request for a team March 31, 2025 21:58
Copy link
Contributor

@mmalerba mmalerba left a comment

Choose a reason for hiding this comment

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

lgtm, but there's an unused import to clean up

@andrewseguin andrewseguin added the action: merge The PR is ready for merge by the caretaker label Apr 1, 2025
@andrewseguin andrewseguin merged commit f8ba137 into angular:main Apr 1, 2025
18 of 22 checks passed
szoboszlaypali pushed a commit to szoboszlaypali/components that referenced this pull request Apr 1, 2025
* refactor: merge m3 mdc and mat tokens

* refactor: missing tokens

* refactor: missing token

* refactor: lint

---------

Co-authored-by: Andrew Seguin <[email protected]>
mistrykaran91 pushed a commit to mistrykaran91/components that referenced this pull request Apr 7, 2025
* refactor: merge m3 mdc and mat tokens

* refactor: missing tokens

* refactor: missing token

* refactor: lint

---------

Co-authored-by: Andrew Seguin <[email protected]>
@angular-automatic-lock-bot
Copy link

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

This action has been performed automatically by a bot.

@angular-automatic-lock-bot angular-automatic-lock-bot bot locked and limited conversation to collaborators May 2, 2025
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
action: merge The PR is ready for merge by the caretaker target: major This PR is targeted for the next major release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants