-
-
Notifications
You must be signed in to change notification settings - Fork 182
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 LoremIpsum import for esm #226
Conversation
1 similar comment
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.
Do you have any idea to avoid such regression in the past? Is there a test we could add for that purpose (in another review)?
package.json
Outdated
@@ -1,6 +1,6 @@ | |||
{ | |||
"name": "fast-check", | |||
"version": "1.7.0", | |||
"version": "1.7.1", |
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.
Please do not bump manually, I will on my side with a specific commit.
@@ -1,18 +1,25 @@ | |||
// tslint:disable-next-line:no-var-requires |
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 think we can remove this line now
I think if we added a headless browser test to make sure it can run in the browser. https://docs.travis-ci.com/user/chrome I may give this a shot this weekend. |
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.
Thanks a lot for the fix :)
I will publish the new version tomorrow morning (Paris time) |
No description provided.