Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Mocha 2.0+ / Minitest 5.19+ compatibility #266

Merged
merged 2 commits into from
Aug 3, 2023
Merged

Mocha 2.0+ / Minitest 5.19+ compatibility #266

merged 2 commits into from
Aug 3, 2023

Conversation

voxik
Copy link
Contributor

@voxik voxik commented Aug 2, 2023

The support for require 'mocha/setup' is long time deprecated and was removed in Mocha 2.0:

freerange/mocha@642a0ff4

The support for `require 'mocha/setup'` is long time deprecated and was
removed in Mocha 2.0:

freerange/mocha@642a0ff4
@voxik
Copy link
Contributor Author

voxik commented Aug 2, 2023

@valtri This soon will be needed in Fedora

The `MiniTest` was renamed to `Minitest`:

minitest/minitest@9a57c52

And the `MiniTest` constant is now loaded just when `MT_COMPAT`
environment variable is set:

minitest/minitest@a2c6c18
@voxik voxik changed the title Mocha 2.0+ compatibility Mocha 2.0+ / Minitest 5.19+ compatibility Aug 3, 2023
@cyu cyu merged commit 555ac46 into cyu:master Aug 3, 2023
10 checks passed
@cyu
Copy link
Owner

cyu commented Aug 3, 2023

Thanks!

# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants