Skip to content

Conversation

@heracle
Copy link
Collaborator

@heracle heracle commented Jul 28, 2021

Add the test files that will be further used inside unit and integration taxonomic tests

@heracle heracle requested review from hmusta and karasikov July 28, 2021 18:07
Base automatically changed from mtg_taxclass_1 to master July 28, 2021 19:58
Copy link
Member

@karasikov karasikov left a comment

Choose a reason for hiding this comment

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

add the code too. Otherwise this data is useless

@heracle
Copy link
Collaborator Author

heracle commented Jul 30, 2021

Thanks for reviewing my PRs! I am trying to split the code into small pieces and to send them in an order that would make sense. I will add these files in another PR with all the integration tests

@karasikov
Copy link
Member

I don't see how these files are used. Please merge to the next PR.

@heracle
Copy link
Collaborator Author

heracle commented Aug 3, 2021

I will merge them to #348 , the PR with the CLI and the integration tests. Thanks!

Signed-off-by: Radu Muntean <[email protected]>
@heracle
Copy link
Collaborator Author

heracle commented Aug 4, 2021

closing this after moving the files to #348

@heracle heracle closed this Aug 4, 2021
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