Skip to content
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 mocha option from colors -> color #45

Open
wants to merge 2 commits into
base: master
Choose a base branch
from
Open

Conversation

ppg
Copy link

@ppg ppg commented Jul 17, 2021

This PR contains:

  • bugfix
  • feature
  • refactor
  • tests
  • documentation
  • metadata

Breaking Changes?

  • yes
  • no

Mocha v7 deprecates useColors in favor of color, and Mocha v8 and onwards don't support useColors. That means this fix of #42 (comment) will work with Mocha v7 and forward, but not Mocha v6 and before.

Please Describe Your Changes

Addresses #42 (comment) to fix the updated mocha option from colors (wrong) to color.

This adds a peerDependencies entry in package.json for "mocha": ">= 7.0.0" to represent the requirement of a mocha version that supports the color option.

@ppg
Copy link
Author

ppg commented Jul 17, 2021

@shellscape This PR addresses #42 (comment) for Mocha v8 and forward support. Per another comment on that PR would it be possible to get this merged and cut into a release?

@SamiSaves SamiSaves mentioned this pull request Jun 3, 2022
8 tasks
@alehuo alehuo mentioned this pull request Mar 21, 2023
8 tasks
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant