-
Notifications
You must be signed in to change notification settings - Fork 26
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
Extracts component styles as Tailwind plugins #18
Conversation
037efce
to
4b918af
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.
This looks pretty good to me (and I'm stoked that we'll be able to use Tailwind!).
4b918af
to
f8d2b3d
Compare
f8d2b3d
to
8c42e1a
Compare
New dependencies detected. Learn more about Socket for GitHub ↗︎
|
8c42e1a
to
b4e5ca1
Compare
const cssAsJs = postcssJs.objectify(root) | ||
|
||
return writeFile( | ||
join(config.buildPath, 'plugins', `${fileParts.name}Component.js`), |
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.
reported by reviewdog 🐶
[semgrep] Detected possible user input going into a path.join
or path.resolve
function. This could possibly lead to a path traversal vulnerability, where the attacker can access arbitrary files stored in the file system. Instead, be sure to sanitize or validate user input first.
Source: https://semgrep.dev/r/javascript.lang.security.audit.path-traversal.path-join-resolve-traversal.path-join-resolve-traversal
Cc @thypon @bcaller
Looks like format is failing - running |
b4e5ca1
to
60450ea
Compare
@@ -5,6 +5,9 @@ const postcss = require('postcss') | |||
|
|||
const cssFiles = ['variables.css'] | |||
|
|||
const genPluginFilePath = (file, config) => | |||
join(config.buildPath, 'plugins', `_${file}.ts`) |
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.
reported by reviewdog 🐶
[semgrep] Detected possible user input going into a path.join
or path.resolve
function. This could possibly lead to a path traversal vulnerability, where the attacker can access arbitrary files stored in the file system. Instead, be sure to sanitize or validate user input first.
Source: https://semgrep.dev/r/javascript.lang.security.audit.path-traversal.path-join-resolve-traversal.path-join-resolve-traversal
Cc @thypon @bcaller
8a79230
to
f74fb10
Compare
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.leo.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-08-23 21:36:39.246926648 +0000
+++ ./tokens/css/variables.css 2023-08-23 21:35:55.610908822 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Wed Aug 23 2023 19:58:56 GMT+0000 (Coordinated Universal Time)
+ * Generated on Wed Aug 23 2023 21:35:55 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
050bec3
to
a2924c2
Compare
a2924c2
to
bde3e4e
Compare
Co-authored-by: Jay Harris <[email protected]>
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.leo.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-08-29 23:28:04.867956166 +0000
+++ ./tokens/css/variables.css 2023-08-29 23:27:24.829616830 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Tue Aug 29 2023 22:01:19 GMT+0000 (Coordinated Universal Time)
+ * Generated on Tue Aug 29 2023 23:27:24 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.leo.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-08-29 23:31:33.441230125 +0000
+++ ./tokens/css/variables.css 2023-08-29 23:31:01.025096186 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Tue Aug 29 2023 22:01:19 GMT+0000 (Coordinated Universal Time)
+ * Generated on Tue Aug 29 2023 23:31:01 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
Does this fix #4? |
It sounds like it might, but I'm not certain. |
7f43ebd
to
98e60f0
Compare
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.leo.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-09-07 18:24:26.083004949 +0000
+++ ./tokens/css/variables.css 2023-09-07 18:23:51.138332856 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Wed Sep 06 2023 21:27:54 GMT+0000 (Coordinated Universal Time)
+ * Generated on Thu Sep 07 2023 18:23:51 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.leo.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-09-07 18:28:27.170340742 +0000
+++ ./tokens/css/variables.css 2023-09-07 18:27:38.742108830 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Wed Sep 06 2023 21:27:54 GMT+0000 (Coordinated Universal Time)
+ * Generated on Thu Sep 07 2023 18:27:38 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
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
98e60f0
to
fbe77c8
Compare
👋 Thanks for Submitting! This PR is available for preview at the link below. ✅ PR tip preview: https://18.pr.nala.bravesoftware.com/ Variables Diff--- ./tokens/css/variables.old.css 2023-09-28 21:46:51.510420868 +0000
+++ ./tokens/css/variables.css 2023-09-28 21:46:10.385932891 +0000
@@ -1,6 +1,6 @@
/**
* Do not edit directly
- * Generated on Thu Sep 28 2023 21:25:54 GMT+0000 (Coordinated Universal Time)
+ * Generated on Thu Sep 28 2023 21:46:10 GMT+0000 (Coordinated Universal Time)
*/
:root {
|
No description provided.