update index.md #148
update index.md #148
Conversation
Made some minor changes in the introduction to better represent shell.
index.md
Outdated
|
|
||
| There are many reasons to learn about the shell. | ||
|
|
||
| - For most bioinformatics tools, you have to use the shell. There is no graphical interface. If you want to work in metagenomics or genomics you're going to need to use the shell. | ||
| - For most bioinformatics tools, you are better off using the shell. There is no graphical interface or even if there is a GUI, it is clunky. If you want to work in metagenomics or genomics you're going to need to use the shell. | ||
| - The shell gives you power. The command line gives you the power to do your work more efficiently and more quickly. When you need to do things tens to hundreds of times, knowing how to use the shell is transformative. |
There was a problem hiding this comment.
Thank you @Sivanandan for this pull request to better represent the shell. Your pull request is highly appreciated. The first part looks good to me! Your second suggestion at line 9 sounds a bit dismissive to me, given that our learners can be expected to be experienced with a number of bioinformatics GUIs (see How not to demotivate your learners). Can you exclude this second part from your pull request?
There was a problem hiding this comment.
Thank you @aschuerch for your review of my pull request. I just finished my instructor training and I can see how I have unconsciously made use of dismissive language. Yes, I will exclude the second part from this request.
This is in response to the review of my original pull request.
|
Please feel free to edit my modified request if you think I may have changed some of the original phrasing. |
|
LGTM! |
|
Is there a reason why the reason |
|
Not really 😃... but when I made the edits, I ended up removing some
original phrasing, just for writing flow. That is why I commented earlier
that you can modify it accordingly so that the original meaning and intent
is not lost.
…On Thu, Jun 28, 2018 at 12:06 PM, Bastian Greshake Tzovaras < ***@***.***> wrote:
Is there a reason why the reason For most bioinformatics tools, you have
to use the shell should be removed? 😃
—
You are receiving this because you were mentioned.
Reply to this email directly, view it on GitHub
<#148 (comment)>,
or mute the thread
<https://github.com/notifications/unsubscribe-auth/AKdn1x3OvUjh3c-iVUEPW73bao1YBvrVks5uBQ0ygaJpZM4UuHOz>
.
--
Siva
|
|
Personally I think it would be nice to keep that, as it might be a strong motivation. @aschuerch what do you think? :) |
|
Agreed |
|
I made the changes to add it back in. |
* update index.md Made some minor changes in the introduction to better represent shell. * Modified index.md This is in response to the review of my original pull request. * re-add motivation
Made some minor changes in the introduction to better represent shell.