-
Notifications
You must be signed in to change notification settings - Fork 4
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
feat(scripts): allow import script to pull using Figma API #2132
Conversation
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## next #2132 +/- ##
==========================================
- Coverage 97.13% 96.96% -0.18%
==========================================
Files 114 114
Lines 2829 2898 +69
Branches 764 781 +17
==========================================
+ Hits 2748 2810 +62
- Misses 77 84 +7
Partials 4 4 ☔ View full report in Codecov by Sentry. |
size-limit report 📦
|
d9f8e5a
to
0eea26d
Compare
@@ -99,7 +99,7 @@ | |||
"jsonfile": "^6.1.0", | |||
"lilconfig": "^3.1.3", | |||
"lodash": "^4.17.21", | |||
"ora": "^8.1.1", | |||
"ora": "5.4.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
note: locking ora to a pre ESM release to avoid import issues and incompatibility with pre v30 of jest
0eea26d
to
f814322
Compare
- handle both downloaded file version and API version - handle input and edge cases (e.g., missing config) - separate and handle parsing of token contents into theme file - guard against committing local edsrc config
f814322
to
6fffc39
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
lgtm! though figma is intimidating lol
Holding merge until release completes to keep the history on |
Yahhhh. adding these classes to make dealing with all the weird data a little more straight-forward... more to come :) |
we also flag instances of mismatches between figma and code, for later fixing (in Figma). Example:
Test Plan: