Disable auto semicolon (#484)

* Disable auto semicolon * Actually disable it
emotion-js · Dec 7, 2017 · 47ec911 · 47ec911
1 parent 6d24323
commit 47ec911
Show file tree

Hide file tree

Showing 3 changed files with 24 additions and 1 deletion.
diff --git a/packages/emotion/src/index.js b/packages/emotion/src/index.js
@@ -13,7 +13,7 @@ export const sheet = new StyleSheet()
 
 // 🚀
 sheet.inject()
-const stylisOptions = { keyframe: false }
+const stylisOptions = { keyframe: false, semicolon: true }
 
 if (process.env.NODE_ENV !== 'production') {
   stylisOptions.compress = false

diff --git a/packages/emotion/test/__snapshots__/css.test.js.snap b/packages/emotion/test/__snapshots__/css.test.js.snap
@@ -334,6 +334,16 @@ exports[`css multiline declaration 1`] = `
 />
 `;
 
+exports[`css multiline selector 1`] = `
+.glamor-0 .my-class:hover .its-child {
+  background: pink;
+}
+
+<div
+  className="glamor-0"
+/>
+`;
+
 exports[`css nested 1`] = `
 .glamor-0 {
   color: yellow;

diff --git a/packages/emotion/test/css.test.js b/packages/emotion/test/css.test.js
@@ -378,6 +378,19 @@ describe('css', () => {
     `
     /* eslint-enable prettier/prettier */
 
+    const tree = renderer.create(<div className={cls1} />).toJSON()
+    expect(tree).toMatchSnapshot()
+  })
+  test('multiline selector', () => {
+    /* eslint-disable prettier/prettier */
+    const cls1 = css`
+      .my-class:hover
+      .its-child {
+        background: pink;
+      }
+    `
+    /* eslint-enable prettier/prettier */
+
     const tree = renderer.create(<div className={cls1} />).toJSON()
     expect(tree).toMatchSnapshot()
   })