Skip to content

New tool: taffy#7826

Open
Maed0x wants to merge 4 commits intogalaxyproject:mainfrom
Maed0x:taffy
Open

New tool: taffy#7826
Maed0x wants to merge 4 commits intogalaxyproject:mainfrom
Maed0x:taffy

Conversation

@Maed0x
Copy link
Copy Markdown
Contributor

@Maed0x Maed0x commented Mar 24, 2026

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)

There are two labels that allow to ignore specific (false positive) tool linter errors:

  • skip-version-check: Use it if only a subset of the tools has been updated in a suite.
  • skip-url-check: Use it if github CI sees 403 errors, but the URLs work.

This PR adds taffy as a new tool (https://github.com/ComparativeGenomicsToolkit/taffy)

@Maed0x
Copy link
Copy Markdown
Contributor Author

Maed0x commented Mar 24, 2026

Linting error mainly consist of missing data types:

  • taf
  • taf.gz
  • tai (MAF/TAF index file used in taffy for efficient random access to specific regions of the alignment)

@SaimMomin12
Copy link
Copy Markdown
Contributor

Linting error mainly consist of missing data types:

  • taf
  • taf.gz
  • tai (MAF/TAF index file used in taffy for efficient random access to specific regions of the alignment)

Can you add the missing datatypes here and target the PR to release_26.0 branch?

Comment thread tools/taffy/macros.xml Outdated
@Maed0x Maed0x requested a review from SaimMomin12 March 24, 2026 13:35
Copy link
Copy Markdown
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.

LGTM! Lets ask for second pair of eyes on this.

@SaimMomin12 SaimMomin12 marked this pull request as ready for review March 26, 2026 14:19
@SaimMomin12 SaimMomin12 requested a review from bgruening March 26, 2026 14:19
@Maed0x
Copy link
Copy Markdown
Contributor Author

Maed0x commented Apr 13, 2026

Hi, are there any updates to this PR?

@SaimMomin12
Copy link
Copy Markdown
Contributor

@bgruening can we merge this?

Comment thread tools/taffy/.shed.yml Outdated
@bgruening
Copy link
Copy Markdown
Member

Yes, I think it can be merged. Thanks @Maed0x

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