Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Fix SpyBOAT wrapper license #4837

Merged
merged 1 commit into from
Oct 10, 2022
Merged

Fix SpyBOAT wrapper license #4837

merged 1 commit into from
Oct 10, 2022

Conversation

nsoranzo
Copy link
Member

@nsoranzo nsoranzo commented Oct 10, 2022 •
edited
Loading

The license attribute refers to the wrapper's license, not the license of the underlying tool(s), see https://docs.galaxyproject.org/en/master/dev/schema.html#attributes

xref. galaxy-iuc/standards#72

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)

@nsoranzo
Fix SpyBOAT wrapper license
a58ce19 
The `license` attribute refers to the wrapper's license, not the
license of the underlying tool(s), see

https://docs.galaxyproject.org/en/master/dev/schema.html#attributes

xref. galaxy-iuc/standards#72
@nsoranzo nsoranzo requested a review from bgruening October 10, 2022 11:43
@nsoranzo nsoranzo merged commit 9b86d7e into galaxyproject:master Oct 10, 2022
@nsoranzo nsoranzo deleted the fix_spyboat_wrapper_license branch October 10, 2022 16:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@bernt-matthias bernt-matthias bernt-matthias approved these changes

@bgruening bgruening bgruening approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@nsoranzo @bgruening @bernt-matthias