Closed Jeong-jj closed 1 month ago
Latest commit: 6d5845a13ca55935449ceccc4cf3bd7d4ee931f8
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
The changes involve the addition of new script commands in the package.json
file for executing tests and watching for changes. Additionally, a series of type tests for VariantSelection
, CompoundVariant
, and PatternOptions
are introduced in TypeScript, enhancing type safety and validation for variant-related structures using Vitest.
Files | Change Summary |
---|---|
packages/css/package.json | Added new script commands: "test": "yarn g:test" and "test:watch": "yarn g:test-watch" . |
packages/css/src/rules/types.ts | Added type tests for VariantSelection<ExampleVariants> , CompoundVariant<Variants> , and PatternOptions<Variants> . |
VariantSelection
and CompoundVariant
added in the main PR.package.json
file, which is relevant since the main PR also modifies the package.json
to add new script commands.publish
script in this PR is related to the changes in the scripts
section of the package.json
in the main PR, indicating a focus on script management.In the land of types, so bright and clear,
Variants dance, with no hint of fear.
Tests now abound, both strong and true,
Ensuring our code will always renew.
With each little check, our types take flight,
A rabbit's delight, in the coding night! 🐇✨
Thank you for using CodeRabbit. We offer it for free to the OSS community and would appreciate your support in helping us grow. If you find it useful, would you consider giving us a shout-out on your favorite social media?
Triggered from https://github.com/mincho-js/mincho/pull/99 by @​Jeong-jj.
Checking if we can fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(61fbf5d716d1762f2940963896b4eb4f4aa7b7d8).
Target branch (main
):
commit c5b0d10038d86c496581e2ca80b3e32cbaed0762 (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Fri Sep 6 20:50:13 2024 +0900
Docs: Introduce link update #90
Pull request (test/rules
):
commit 61fbf5d716d1762f2940963896b4eb4f4aa7b7d8 (pull_request/test/rules)
Author: JeongJun <rgfdds98@gmail.com>
Date: Sun Sep 8 23:42:07 2024 +0900
Test: origin PatternOptions type test
Can't fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(61fbf5d716d1762f2940963896b4eb4f4aa7b7d8). main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) is not a direct ancestor of test/rules
(61fbf5d716d1762f2940963896b4eb4f4aa7b7d8). Branches appear to have diverged at ef27468caa714eea254f9b9c6563ba2a5483a135:
* 61fbf5d716d1762f2940963896b4eb4f4aa7b7d8 Test: origin PatternOptions type test
| * c5b0d10038d86c496581e2ca80b3e32cbaed0762 Docs: Introduce link update #90
|/
* ef27468caa714eea254f9b9c6563ba2a5483a135 Feat: Rules - Basic porting for CSS rules with vanilla extract recipe #94
commit ef27468caa714eea254f9b9c6563ba2a5483a135
Author: alstjr7375 <alstjr7375@daum.net>
Date: Mon Sep 2 15:54:16 2024 +0900
Feat: Rules - Basic porting for CSS rules with vanilla extract recipe #94
Rebase locally, and then force push to test/rules
.
Triggered from https://github.com/mincho-js/mincho/pull/99 by @​Jeong-jj.
Checking if we can fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(b8cf197458a0f81ca8a4813b4a740a71a65d46a4).
Target branch (main
):
commit c5b0d10038d86c496581e2ca80b3e32cbaed0762 (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Fri Sep 6 20:50:13 2024 +0900
Docs: Introduce link update #90
Pull request (test/rules
):
commit b8cf197458a0f81ca8a4813b4a740a71a65d46a4 (pull_request/test/rules)
Author: JeongJun Jo <rgfdds98@gmail.com>
Date: Wed Sep 18 21:04:36 2024 +0900
Test: origin CompoundVariant Type Test
Can't fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(b8cf197458a0f81ca8a4813b4a740a71a65d46a4). main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) is not a direct ancestor of test/rules
(b8cf197458a0f81ca8a4813b4a740a71a65d46a4). Branches appear to have diverged at ef27468caa714eea254f9b9c6563ba2a5483a135:
* b8cf197458a0f81ca8a4813b4a740a71a65d46a4 Test: origin CompoundVariant Type Test
* 95f9ad32707f0d3c6669e497a2d5e683e79e7806 Test: VariantSelection Type Test
* fff002291efa054af78f851eede369cb58751aca Test: origin PatternOptions type test
| * c5b0d10038d86c496581e2ca80b3e32cbaed0762 Docs: Introduce link update #90
|/
* ef27468caa714eea254f9b9c6563ba2a5483a135 Feat: Rules - Basic porting for CSS rules with vanilla extract recipe #94
commit ef27468caa714eea254f9b9c6563ba2a5483a135
Author: alstjr7375 <alstjr7375@daum.net>
Date: Mon Sep 2 15:54:16 2024 +0900
Feat: Rules - Basic porting for CSS rules with vanilla extract recipe #94
Rebase locally, and then force push to test/rules
.
Triggered from https://github.com/mincho-js/mincho/pull/99 by @​Jeong-jj.
Checking if we can fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(3e0f0be2e703a92b55e2dcefdc08f645c1e1b582).
Target branch (main
):
commit c5b0d10038d86c496581e2ca80b3e32cbaed0762 (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Fri Sep 6 20:50:13 2024 +0900
Docs: Introduce link update #90
Pull request (test/rules
):
commit 3e0f0be2e703a92b55e2dcefdc08f645c1e1b582 (pull_request/test/rules)
Author: JeongJun Jo <rgfdds98@gmail.com>
Date: Wed Sep 18 21:04:36 2024 +0900
Test: origin CompoundVariant Type Test
It is possible to fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(3e0f0be2e703a92b55e2dcefdc08f645c1e1b582). If you have write access to the target repository, you can add a comment with /fast-forward
to fast forward main
to test/rules
.
Triggered from https://github.com/mincho-js/mincho/pull/99 by @​Jeong-jj.
Checking if we can fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(32d306a2732cc273f181c0bc1a8f0536aa33ad44).
Target branch (main
):
commit c5b0d10038d86c496581e2ca80b3e32cbaed0762 (HEAD -> main, origin/main, origin/HEAD)
Author: alstjr7375 <alstjr7375@daum.net>
Date: Fri Sep 6 20:50:13 2024 +0900
Docs: Introduce link update #90
Pull request (test/rules
):
commit 32d306a2732cc273f181c0bc1a8f0536aa33ad44 (pull_request/test/rules)
Author: JeongJun Jo <rgfdds98@gmail.com>
Date: Wed Sep 18 23:25:38 2024 +0900
Fix: adding a description of the intended type error
It is possible to fast forward main
(c5b0d10038d86c496581e2ca80b3e32cbaed0762) to test/rules
(32d306a2732cc273f181c0bc1a8f0536aa33ad44). If you have write access to the target repository, you can add a comment with /fast-forward
to fast forward main
to test/rules
.
@coderabbitai review
Triggered from https://github.com/mincho-js/mincho/pull/99 by @​black7375.
Checking if we can fast forward main
(8e58ddb31fce27bc55fbfa18542df6a33daa1349) to test/rules
(6d5845a13ca55935449ceccc4cf3bd7d4ee931f8).
Target branch (main
):
commit 8e58ddb31fce27bc55fbfa18542df6a33daa1349 (HEAD -> main, origin/main, origin/HEAD)
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date: Sat Sep 28 13:17:12 2024 +0000
Chore(deps): Bump micromatch in the npm_and_yarn group
Bumps the npm_and_yarn group with 1 update: [micromatch](https://github.com/micromatch/micromatch).
Updates `micromatch` from 4.0.7 to 4.0.8
- [Release notes](https://github.com/micromatch/micromatch/releases)
- [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md)
- [Commits](https://github.com/micromatch/micromatch/compare/4.0.7...4.0.8)
---
updated-dependencies:
- dependency-name: micromatch
dependency-type: indirect
dependency-group: npm_and_yarn
...
Signed-off-by: dependabot[bot] <support@github.com>
Pull request (test/rules
):
commit 6d5845a13ca55935449ceccc4cf3bd7d4ee931f8 (pull_request/test/rules)
Author: JeongJun <rgfdds98@gmail.com>
Date: Sun Sep 29 00:16:49 2024 +0900
Test: origin PatternOptions type test
It is possible to fast forward main
(8e58ddb31fce27bc55fbfa18542df6a33daa1349) to test/rules
(6d5845a13ca55935449ceccc4cf3bd7d4ee931f8). If you have write access to the target repository, you can add a comment with /fast-forward
to fast forward main
to test/rules
.
/fast-forward
Triggered from https://github.com/mincho-js/mincho/pull/99#issuecomment-2380705980 by @​black7375.
Trying to fast forward main
(8e58ddb31fce27bc55fbfa18542df6a33daa1349) to test/rules
(6d5845a13ca55935449ceccc4cf3bd7d4ee931f8).
Target branch (main
):
commit 8e58ddb31fce27bc55fbfa18542df6a33daa1349 (HEAD -> main, origin/main, origin/HEAD)
Author: dependabot[bot] <49699333+dependabot[bot]@users.noreply.github.com>
Date: Sat Sep 28 13:17:12 2024 +0000
Chore(deps): Bump micromatch in the npm_and_yarn group
Bumps the npm_and_yarn group with 1 update: [micromatch](https://github.com/micromatch/micromatch).
Updates `micromatch` from 4.0.7 to 4.0.8
- [Release notes](https://github.com/micromatch/micromatch/releases)
- [Changelog](https://github.com/micromatch/micromatch/blob/master/CHANGELOG.md)
- [Commits](https://github.com/micromatch/micromatch/compare/4.0.7...4.0.8)
---
updated-dependencies:
- dependency-name: micromatch
dependency-type: indirect
dependency-group: npm_and_yarn
...
Signed-off-by: dependabot[bot] <support@github.com>
Pull request (test/rules
):
commit 6d5845a13ca55935449ceccc4cf3bd7d4ee931f8 (pull_request/test/rules)
Author: JeongJun <rgfdds98@gmail.com>
Date: Sun Sep 29 00:16:49 2024 +0900
Test: origin PatternOptions type test
Fast forwarding main
(8e58ddb31fce27bc55fbfa18542df6a33daa1349) to test/rules
(6d5845a13ca55935449ceccc4cf3bd7d4ee931f8).
$ git push origin 6d5845a13ca55935449ceccc4cf3bd7d4ee931f8:main
To https://github.com/mincho-js/mincho.git
8e58ddb..6d5845a 6d5845a13ca55935449ceccc4cf3bd7d4ee931f8 -> main
Description
The core types test about Rules corresponding to Recipe.
Related Issue
93
Summary by CodeRabbit
New Features
Bug Fixes
Additional context
It remain about
RecipeClassNames
,RuntimeFn
Type Test (24.09.18). I'm still thinking about type tests about these. I'll finish soon.Checklist