Skip to content

Conversation

@james-toussaint
Copy link
Contributor

@james-toussaint james-toussaint commented Jan 9, 2025

Feature:
image

@james-toussaint james-toussaint self-assigned this Jan 9, 2025
@james-toussaint james-toussaint marked this pull request as ready for review January 9, 2025 14:41
Copy link
Contributor

@gfournieriExec gfournieriExec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for this improvement


# Setup coverage⁠

> In order for Matchstick to check which handlers are being run, those handlers need to be exported from the test file.
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice

Copy link
Member

@zguesmi zguesmi left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice!

Copy link
Contributor

@Le-Caignec Le-Caignec left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@james-toussaint james-toussaint merged commit 51722d5 into develop Jan 13, 2025
4 checks passed
@james-toussaint james-toussaint deleted the feature/coverage branch January 13, 2025 14:12
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.

4 participants