Skip to content
This repository has been archived by the owner on Feb 2, 2024. It is now read-only.

New pattern to update module #5

Merged
merged 7 commits into from
Oct 15, 2023
Merged

New pattern to update module #5

merged 7 commits into from
Oct 15, 2023

Conversation

vlopezferrando
Copy link
Contributor

No description provided.

Copy link
Contributor

@morgante morgante left a comment

Choose a reason for hiding this comment

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

Looks fine. Ideally we would include/add a test case.

@vlopezferrando
Copy link
Contributor Author

I have added a test. I don't think there's a standard way to test multi-file patterns, so I added a directory with some fixtures with input and expected output and a script to run it.

@morgante morgante merged commit 0fe322d into getgrit:main Oct 15, 2023
1 check passed
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants