-
Notifications
You must be signed in to change notification settings - Fork 119
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
chore: 🤖 Yarn upgrade, tsdx and docz version bump #36
Conversation
ba64a80
to
39bce2b
Compare
@@ -31,7 +31,7 @@ export function MultipleLineEditor(props: MultipleLineEditorProps) { | |||
error={errors.length > 0} | |||
disabled={disabled} | |||
value={value || ''} | |||
onChange={(e: React.ChangeEvent<HTMLInputElement>) => { | |||
onChange={(e: React.ChangeEvent<HTMLTextAreaElement>) => { |
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.
👍
package.json
Outdated
@@ -77,7 +81,7 @@ | |||
"remark-preset-lint-recommended": "^3.0.3", | |||
"remark-validate-links": "^9.0.1", | |||
"start-server-and-test": "^1.10.6", | |||
"tsdx": "0.11.0", | |||
"tsdx": "0.12.0", |
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 is 0.12.3 available already. do we need 0.12.0?
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.
Good point, I'll look into that.
@@ -1,3 +1,4 @@ | |||
import noop from 'lodash-es/noop'; |
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.
I'd import `import noop from 'lodash/noop' as we do in other places. TSDX transforms it to tree-shakable thing automatically
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.
I already tried that but I got
SyntaxError: Unexpected token 'export'
> 1 | import noop from 'lodash-es/noop';
| ^
2 | import React, { useState, useCallback } from 'react';
3 | import { css } from 'emotion';
4 | import tokens from '@contentful/forma-36-tokens';
While already spending too much time on all of this chore stuff I decided not to look into it. If you can point me at how to solve this without lots of trial and error let me know and I can do.
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.
I'd leave it as it is then :)
- To use .jsx files in the upcoming rich text field editor we need tsdx 0.12 together with the workaround mentioned on jaredpalmer/tsdx#443 - Use docz 1.3.2 - Ran `yarn upgrade`
To use .jsx files in the upcoming rich text field editor we need tsdx 0.12 together with the workaround mentioned on jaredpalmer/tsdx#443