Skip to content

Update meme and fix tests#2744

Merged
bgruening merged 2 commits intogalaxyproject:masterfrom
gregvonkuster:meme
Dec 11, 2019
Merged

Update meme and fix tests#2744
bgruening merged 2 commits intogalaxyproject:masterfrom
gregvonkuster:meme

Conversation

@gregvonkuster
Copy link
Copy Markdown
Contributor

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)

@gregvonkuster
Copy link
Copy Markdown
Contributor Author

Is it best to keep tool suites like this all in the same repo, or should I move each tool to a separate repo? I tend to prefer the latter, but I'll go with consensus.

@bgruening
Copy link
Copy Markdown
Member

@gregvonkuster I'm not sure what you mean with repo. But a classical suite are many tools that are in one folder, share one-or-many macro files and are demultiplexed into many TS repos.

@gregvonkuster
Copy link
Copy Markdown
Contributor Author

Ok, thanks, I'll keep it this way. I'll add the fixes for the 2 additional tools - should have done that in the initial commit. ;)

@bgruening bgruening merged commit 86a94f4 into galaxyproject:master Dec 11, 2019
@bgruening
Copy link
Copy Markdown
Member

Thanks @gregvonkuster!

@gregvonkuster gregvonkuster deleted the meme branch April 25, 2020 12:59
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