Skip to content

refactor(material/core): split token utils#29498

Merged
crisbeto merged 2 commits intoangular:mainfrom
andrewseguin:split-util-deps
Jul 29, 2024
Merged

refactor(material/core): split token utils#29498
crisbeto merged 2 commits intoangular:mainfrom
andrewseguin:split-util-deps

Conversation

@andrewseguin
Copy link
Copy Markdown
Contributor

@andrewseguin andrewseguin commented Jul 27, 2024

No-op change that moves a subset of token-utils.scss functions into their own util file called token-definition-utils.scss. This is necessary to avoid circular deps in any future changes where _token-utils.scss needs to depend on m3-tokens.scss (for system fallbacks)

@andrewseguin andrewseguin force-pushed the split-util-deps branch 4 times, most recently from 6baf06c to ab3ff63 Compare July 27, 2024 21:47
@andrewseguin andrewseguin marked this pull request as ready for review July 28, 2024 12:37
@andrewseguin andrewseguin requested a review from a team as a code owner July 28, 2024 12:37
@andrewseguin andrewseguin requested review from amysorto and crisbeto and removed request for a team July 28, 2024 12:37
@andrewseguin andrewseguin added action: merge The PR is ready for merge by the caretaker target: patch This PR is targeted for the next patch release labels Jul 28, 2024
@andrewseguin andrewseguin removed the request for review from amysorto July 28, 2024 16:20
@angular-robot angular-robot bot requested a review from crisbeto July 29, 2024 02:41
@crisbeto crisbeto merged commit 0a6b3ea into angular:main Jul 29, 2024
crisbeto pushed a commit that referenced this pull request Jul 29, 2024
* refactor(material/core): use util to build token vars

* rename to just token-definition

(cherry picked from commit 0a6b3ea)
@angular-automatic-lock-bot
Copy link
Copy Markdown

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 Aug 29, 2024
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: patch This PR is targeted for the next patch release

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants