Skip to content

Conversation

@MatiPl01
Copy link
Member

@MatiPl01 MatiPl01 commented Oct 14, 2025

Summary

This PR relocates style prop value preprocessors from the css dir to the common dir. It also moves all dependent code, like some utils and types.

@MatiPl01 MatiPl01 self-assigned this Oct 14, 2025
@MatiPl01 MatiPl01 force-pushed the @matipl01/move-style-processors-to-common branch from 96c99ca to f753a9f Compare October 14, 2025 18:15
Base automatically changed from @matipl01/move-some-types-to-common-dir to main October 15, 2025 10:32
@MatiPl01 MatiPl01 force-pushed the @matipl01/move-style-processors-to-common branch from 6e05498 to d94afa2 Compare October 21, 2025 16:55
@MatiPl01 MatiPl01 changed the base branch from main to @matipl01/unify-color-processing October 21, 2025 16:56
Copy link
Collaborator

@tjzel tjzel left a comment

Choose a reason for hiding this comment

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

All looks well!

@MatiPl01 MatiPl01 marked this pull request as ready for review October 21, 2025 21:39
@MatiPl01 MatiPl01 force-pushed the @matipl01/move-style-processors-to-common branch from 8be38a9 to cfa853b Compare October 21, 2025 22:05
Copy link
Contributor

@patrycjakalinska patrycjakalinska left a comment

Choose a reason for hiding this comment

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

lgtm ✨

@MatiPl01 MatiPl01 force-pushed the @matipl01/unify-color-processing branch from 37c3028 to 422da34 Compare October 22, 2025 09:32
Base automatically changed from @matipl01/unify-color-processing to main October 23, 2025 08:56
@MatiPl01 MatiPl01 force-pushed the @matipl01/move-style-processors-to-common branch from cfa853b to 6e52197 Compare October 23, 2025 08:58
@MatiPl01 MatiPl01 added this pull request to the merge queue Oct 23, 2025
Merged via the queue into main with commit d700021 Oct 23, 2025
12 checks passed
@MatiPl01 MatiPl01 deleted the @matipl01/move-style-processors-to-common branch October 23, 2025 09:53
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants