Skip to content

Conversation

@pchoisel
Copy link
Collaborator

No description provided.

@pchoisel pchoisel requested a review from finetjul January 15, 2024 12:32
Copy link
Member

@finetjul finetjul left a comment

Choose a reason for hiding this comment

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

LGTM

@smjoshiatglobus
Copy link
Contributor

Thank you for doing this!
Just nitpicking... would scrape be a better action verb than scrap for the new file and method?
English is not my first language, and I have made worse mistakes myself, multiple times! I rely on spell-checkers, but they may not catch this one, since both are valid words.

@smjoshiatglobus
Copy link
Contributor

PR #64 should fix (or at least suppress) the unit test failure.

@pchoisel pchoisel merged commit 3ffe350 into KitwareMedical:master Feb 26, 2024
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.

3 participants