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

Typescript port #38

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

Typescript port #38

wants to merge 2 commits into from

Conversation

imdfl
Copy link

@imdfl imdfl commented Jul 15, 2018

Hi,

I used mutation-summary in a typescript project, found it useful and decided to integrate with it. The port was the easiest way for me to understand my way around and fully interface with the code in other typescript files.
I probably haven't got the basic environment to run your tests and all, so my only proof that the port works, is that it works for me. Hope you find it useful. I added one small feature that I needed, a callback in the MS constructor params that can filter summaries (or rather, nodes) before producing the summary.

Cheers,

David

imdfl added 2 commits July 15, 2018 23:49
Changes:
Added a configuration option for a filter callback to ignore some mutations
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