Skip to content

Update tools in tutorial to the newest version (dada-16S) #6189

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.

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

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

SantaMcCloud
Copy link

  • Update multiple tool version to the newest.
  • No changes to any grafics since the main tools didnt had any updates

@SantaMcCloud
Copy link
Author

Hello,

This is my first time updating a tutorial. I did choose this one since there are no heavy changes needed. I just want to do it the first time to know how to do it since i should update a bigger tutorial which might need a bit more work and i thougth doing this first is a good start in general into this topic.

Let me know if i might did done something wrong or if i miss something!

Copy link
Member

@bebatut bebatut left a comment

Choose a reason for hiding this comment

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

Thanks @SantaMcCloud for your PR here. The changes in the tutorial look good. Could you apply the same changes in the .ga? Thanks a lot!

@@ -851,7 +851,7 @@ We will construct a simple sample table from the information encoded in the file
>
> We will now extract from the names the factors:
>
> 2. {% tool [Replace Text in entire line](toolshed.g2.bx.psu.edu/repos/bgruening/text_processing/tp_replace_in_line/1.1.2) %}
> 2. {% tool [Replace Text in entire line](toolshed.g2.bx.psu.edu/repos/bgruening/text_processing/tp_replace_in_line/9.5+galaxy2) %}
Copy link
Member

Choose a reason for hiding this comment

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

Could you also update all tool versions in the .ga? Thanks a lot!

Copy link
Author

Choose a reason for hiding this comment

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

oh yes i thought when i downlaod the updated .ga and copy paste it that this all will be updated

Copy link
Author

Choose a reason for hiding this comment

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

oh wait arent all tools updated? it seems so when i had a second look at it? @bebatut

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants