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
fix(deps): Migrate to react-codemirror2 #75
Conversation
1e41851
to
39f38f4
Compare
95e3388
to
d6f8e94
Compare
cache: | ||
yarn: true | ||
directories: | ||
- ~/.cache/Cypress |
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.
See cypress docs
{} | ||
{ | ||
"defaultCommandTimeout": 10000, | ||
"video": false |
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.
Disabling video as we dont publish the assets anywhere so its a waste. Also bumping the timeout as the execution time was inconsistent.
frames={frames} | ||
frameComponent={frameComponent} | ||
/> | ||
<Preview code={renderCode} themes={themes} frames={frames} /> |
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.
Why are frameComponent
and components
not passed anymore?
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.
They are unused.
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.
Nice one. Thanks heaps for doing this 👏
Migrates from
react-codemirror
toreact-codemirror2
.Also upgrading cypress and switching CI over to
yarn
so it honours the lock file.Closes #32.