-
-
Notifications
You must be signed in to change notification settings - Fork 1.1k
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
Change build system and update Jest #747
Changes from 28 commits
e9315c6
ef6dd5c
2ee93eb
1a46819
46b2750
f2f153f
df609e9
f9c975d
c74a39a
21d2b01
95840eb
6e1256c
c5cabe5
1b8e73b
541b1cd
edf6af3
eea5e74
4decc39
86c95dc
b70004d
42f98b8
157a7a0
dad28a7
54acf9f
3d4f99a
157bbec
ad8d1ad
22f1907
43eb9e3
d31ded1
6f89776
44e35e9
29887b9
7ae8d49
200495c
3a0fe42
07aaace
6d552a4
6d9f93b
cb4e562
ec46278
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 |
---|---|---|
@@ -1,7 +1,6 @@ | ||
/coverage | ||
/demo/dist | ||
dist/ | ||
lib/ | ||
node_modules/ | ||
*.log | ||
.idea | ||
|
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,8 +1,8 @@ | ||
[build] | ||
command = "npm run build && npm run build:site" | ||
publish = "packages/site/public" | ||
publish = "site/public" | ||
|
||
[build.environment] | ||
NODE_VERSION = "8" | ||
YARN_VERSION = "1.3.2" | ||
NODE_VERSION = "10" | ||
YARN_VERSION = "1.7.0" | ||
YARN_FLAGS = "--frozen-lockfile" |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
// this can be removed in emotion@10 | ||
module.exports = require('babel-plugin-emotion').macros.styled | ||
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -0,0 +1,2 @@ | ||
// this can be removed in emotion@10 | ||
module.exports = require('babel-plugin-emotion').macros.emotion |
Original file line number | Diff line number | Diff line change |
---|---|---|
@@ -1,9 +1,7 @@ | ||
require('babel-register') | ||
const path = require('path') | ||
require('module-alias').addAliases({ | ||
'emotion-utils': path.join(__dirname, '../../../emotion-utils/src'), | ||
'react-emotion/macro': path.join(__dirname, '../../src/macro-styled'), | ||
'emotion/macro': path.join(__dirname, '../../src/macro') | ||
'babel-plugin-emotion': path.join(__dirname, '../../src') | ||
}) | ||
|
||
module.exports = require('babel-plugin-macros') |
Original file line number | Diff line number | Diff line change |
---|---|---|
|
@@ -11,11 +11,7 @@ | |
"types" | ||
], | ||
"scripts": { | ||
"build": "npm-run-all clean rollup", | ||
"clean": "rimraf dist", | ||
"test:typescript": "dtslint types", | ||
"rollup": "rollup -c ../../rollup.config.js", | ||
"watch": "rollup -c ../../rollup.config.js --watch" | ||
"test:typescript": "dtslint types" | ||
}, | ||
"dependencies": { | ||
"@emotion/is-prop-valid": "^0.6.1" | ||
|
@@ -26,10 +22,7 @@ | |
"devDependencies": { | ||
"@types/react": "16.0.16", | ||
"dtslint": "^0.3.0", | ||
"npm-run-all": "^4.0.2", | ||
"prop-types": "^15.6.1", | ||
"rimraf": "^2.6.1", | ||
"rollup": "^0.60.1" | ||
"prop-types": "^15.6.1" | ||
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. seems weird to have prop-types in devDeps, are they needed here? 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. It was already there. It's probably not strictly necessary but it was probably added there because it's a peerDependency and it's needed in dev. If you think it should be removed them I'll remove it but IMO, it doesn't really matter. 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. peer dep thing makes sense 👍 |
||
}, | ||
"author": "Kye Hohenberger", | ||
"homepage": "https://emotion.sh", | ||
|
@@ -44,5 +37,6 @@ | |
], | ||
"bugs": { | ||
"url": "https://github.com/emotion-js/emotion/issues" | ||
} | ||
}, | ||
"umd:main": "./dist/emotion.umd.min.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.
whats the reason behind those 2 "proxy" files?
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.
In case people have custom instances of emotion and they're using the macros with them since this they were previously imported via
babel-plugin-emotion/lib/macro
andbabel-plugin-emotion/lib/macro-styled
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.
gotcha, maybe add a comment that this can be removed with next major?