-
Notifications
You must be signed in to change notification settings - Fork 8.3k
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
Enable CSS-in-JS styling with emotion
#98157
Changes from all commits
778627a
196bf60
b60a64e
d596827
db03924
fd81944
014f2f0
f457a73
aceaed2
255a812
df51fff
d815ad9
59ac889
c11ebb5
361e2f1
3bfddfd
c877306
cd45fda
6b3d432
b008566
f811056
985577b
1dda574
c777224
3972208
8bdfd6b
cc91272
f6a1e7e
47455f1
4a93b5b
b5d2633
a99dc6a
145a606
7547d66
cbf4c94
b464828
447e80e
File filter
Filter by extension
Conversations
Jump to
Diff view
Diff view
There are no files selected for viewing
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -46,3 +46,14 @@ export async function transformFileWithBabel(file: File) { | |
file.extname = '.js'; | ||
transformedFiles.add(file); | ||
} | ||
|
||
/** | ||
* Synchronized regex list of files that use `styled-components`. | ||
* Used by `kbn-babel-preset` and `elastic-eslint-config-kibana`. | ||
*/ | ||
export const USES_STYLED_COMPONENTS = [ | ||
/packages[\/\\]kbn-ui-shared-deps[\/\\]/, | ||
/src[\/\\]plugins[\/\\](data|kibana_react)[\/\\]/, | ||
/x-pack[\/\\]plugins[\/\\](apm|beats_management|cases|fleet|infra|lists|observability|osquery|security_solution|timelines|uptime)[\/\\]/, | ||
/x-pack[\/\\]test[\/\\]plugin_functional[\/\\]plugins[\/\\]resolver_test[\/\\]/, | ||
There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. Have you created a meta issue to ping codeowners of these plugins to migrate from There was a problem hiding this comment. Choose a reason for hiding this commentThe reason will be displayed to describe this comment to others. Learn more. I have not. Thanks for the reminder |
||
]; |
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -6,7 +6,8 @@ | |
"types": [ | ||
"node", | ||
"jest", | ||
"react" | ||
"react", | ||
"@emotion/react/types/css-prop" | ||
] | ||
}, | ||
"include": [ | ||
|
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.
@mistic does this make sense to you? Or do we think it's one of dependency ordering issues?
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.
There was some related discussion in #99382 (comment)
Basically, modifying
tsconfig.base.json
makes this a dependency almost everywhereThere 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.
yeah it makes sense as we are inheriting from the
tsconfig.base.json
here and we don't override the type definitions in tsconfig file of that package. probably we should do it