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.

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

05-cmdline: do not link to a page about kill #228

Merged
merged 1 commit into from
Sep 21, 2016

Conversation

keszybz
Copy link
Contributor

@keszybz keszybz commented Aug 12, 2016

There is no need to refer to search through ps aux output, and the
page that was linked wasn't even particularly useful, since 'kill %'
works nicely under most shells. Linked page also suggested using
ctrl-alt-esc, which (I'm guessing) invokes something equivalent to
xkill in some graphical environments. Using this would likely
terminate the terminal or Rstudio, which is definitely not what we
want.

There is no need to refer to search through ps aux output, and the
page that was linked wasn't even particularly useful, since 'kill %'
works nicely under most shells. Linked page also suggested using
ctrl-alt-esc, which (I'm guessing) invokes something equivalent to
xkill in some graphical environments. Using this would likely
terminate the terminal or Rstudio, which is definitely not what we
want.
@chendaniely
Copy link
Contributor

chendaniely commented Sep 21, 2016

thanks @keszybz for the changes and contribution!

Next time you just need to submit changes for the .Rmd file. We render and build the changes on our end.

@chendaniely chendaniely merged commit 7e584a7 into swcarpentry:gh-pages Sep 21, 2016
rgaiacs added a commit to rgaiacs/swc-r-novice-inflammation that referenced this pull request Apr 17, 2018
Some commits from (lesson-example/186) Assorted minor fixes to spelling, whitespace, hyperlinks & sorting
zkamvar pushed a commit that referenced this pull request May 1, 2023
05-cmdline: do not link to a page about kill
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.

None yet

2 participants