-
Notifications
You must be signed in to change notification settings - Fork 58
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
Update mocaccino to v6 and mocha to v9 #276
Conversation
@@ -85,7 +85,7 @@ | |||
"devDependencies": { | |||
"@studio/changes": "^2.0.1", | |||
"eslint": "^7.13.0", | |||
"mocha": "^8.4.0", | |||
"mocha": "^9.1.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.
Not sure if this your intention in this PR, but this is the Mocha version used by the tests in this repo. Mochify itself will still be on version 8 as per the version of mocaccino that is used. Is that the idea here?
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 wanted to check if mochify.js would be compatible with mocha v9, as per mantoni/mocaccino.js#38 (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.
I added another commit to also update mocaccino (temporarily to my fork) to see if there would be any conflicts with mocha.
f10aa3f
to
b894dbd
Compare
b894dbd
to
0fa2bf1
Compare
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.
LGTM. Are you ok if we make this 9.0.0
@mantoni ?
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.
@m90 Go! ❤️
This is released as |
Pulls in mocaccino v6 and mocha v9. Mocha v9 is used for internal testing of mochify.
Depends on mantoni/mocaccino.js#38 to update mocaccino versions.