Skip to content

Conversation

@Smkingsize
Copy link
Contributor

@Smkingsize Smkingsize commented Nov 27, 2025

FOR CONTRIBUTOR:

  • I have read the CONTRIBUTING.md document and this tool is appropriate for the tools-iuc repo.
  • License permits unrestricted use (educational + commercial)
  • This PR adds a new tool or tool collection
  • This PR updates an existing tool or tool collection
  • This PR does something else (explain below)

@SaimMomin12 SaimMomin12 mentioned this pull request Nov 27, 2025
@SaimMomin12 SaimMomin12 changed the title add Sourmash [WIP] New tool addition: Sourmash tool suite Nov 27, 2025
@SaimMomin12 SaimMomin12 self-requested a review November 27, 2025 13:22
@SaimMomin12
Copy link
Contributor

@Smkingsize The .shed.yml seems to be empty. Could you re-check?

Copy link
Contributor

@SaimMomin12 SaimMomin12 left a comment

Choose a reason for hiding this comment

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

Some comments inline. Would be nice if we use the following structure instead of individual folders
-- sourmash
-- test-data
-- sourmash_plot.xml
-- sourmash_sketch.xml
-- macros.xml
-- shed.yml (This needs a suite defination)

added some macros
added initial .shed.yml file (not final)
added better assers
removed too big files
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