swcarpentry / shell-novice

The Unix Shell
http://swcarpentry.github.io/shell-novice/
Other
373 stars 969 forks source link

Update remove-unneeded-files sol'n 4 in lesson 4 (issue #1415) #1416

Closed sdrobert closed 11 months ago

sdrobert commented 11 months ago

This PR is follows up issue #1415.

In the removing-unneeded-files question of lesson 4, the description of the 4th solution

The shell would expand `*.*` to match all files with any extension, so this command would delete all files

has been replaced with

The shell expands `*.*` to match all filenames containing at least one `.`, including the processed files (`.txt`) *and* raw files (`.dat`)

Note that the replacement text is slightly different from that suggested in #1415. I can easily use that text if the maintainers would prefer.

Best, Sean

github-actions[bot] commented 11 months ago

Thank you!

Thank you for your pull request :smiley:

:robot: This automated message can help you check the rendered files in your submission for clarity. If you have any questions, please feel free to open an issue in {sandpaper}.

If you have files that automatically render output (e.g. R Markdown), then you should check for the following:

Rendered Changes

:mag: Inspect the changes: https://github.com/swcarpentry/shell-novice/compare/md-outputs..md-outputs-PR-1416

The following changes were observed in the rendered markdown documents:

 04-pipefilter.md | 4 ++--
 md5sum.txt       | 2 +-
 2 files changed, 3 insertions(+), 3 deletions(-)
What does this mean? If you have source files that require output and figures to be generated (e.g. R Markdown), then it is important to make sure the generated figures and output are reproducible. This output provides a way for you to inspect the output in a diff-friendly manner so that it's easy to see the changes that occur due to new software versions or randomisation.

:stopwatch: Updated at 2023-07-19 07:20:15 +0000