Skip to content

Conversation

@KRRT7
Copy link
Contributor

@KRRT7 KRRT7 commented Sep 24, 2025

No description provided.

@second-ed
Copy link
Owner

Hey thanks for this, it looks good, do you have a minimal test example you could add either on here or in a mock_data/mock_case_9/src/case_9 directory and add the expected result to a parametrized case in test_run_process. Let me know if you need a hand with setting up the test or anything!

@KRRT7
Copy link
Contributor Author

KRRT7 commented Sep 28, 2025

I've added the test case, let me know what you think @second-ed

Copy link
Owner

@second-ed second-ed left a comment

Choose a reason for hiding this comment

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

Awesome, looks great - thanks for contributing!

@second-ed second-ed merged commit 2a2aeca into second-ed:dev Sep 29, 2025
1 check passed
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.

2 participants