refactor: replaces chai with node:assert - Githubissues

sverweij / state-machine-cat

write beautiful state charts :scream_cat:

https://state-machine-cat.js.org

MIT License

802 stars 45 forks source link

refactor: replaces chai with node:assert #199

Closed sverweij closed 1 year ago

sverweij commented 1 year ago

Description

replaces chai with node:assert as assertion library

Motivation and Context

less dependencies === less worries - and node:assert is built into nodejs anyway

How Has This Been Tested?

[x] green ci

Types of changes

[ ] Bug fix (non-breaking change which fixes an issue)
[ ] Documentation only change
[x] Refactor (non-breaking change which fixes an issue without changing functionality)
[ ] New feature (non-breaking change which adds functionality)
[ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist

[x] :book:
- My change doesn't require a documentation update, or ...
- it does and I have updated it
[x] :balance_scale:
- The contribution will be subject to The MIT license, and I'm OK with that.
- The contribution is my own original work.
- I am ok with the stuff in CONTRIBUTING.md.

codeclimate[bot] commented 1 year ago

Code Climate has analyzed commit e91bf445 and detected 0 issues on this pull request.

The test coverage on the diff in this pull request is 100.0% (100% is the threshold).

This pull request will bring the total coverage in the repository to 100.0% (0.0% change).

View more on Code Climate.

socket-security[bot] commented 1 year ago

Removed dependencies detected. Learn more about Socket for GitHub ↗︎

🚮 Removed packages: @types/chai@4.3.5, @types/lodash@4.14.196, chai@4.3.7