search

swcarpentry / shell-novice

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

Do not overwrite a file that is used later #1363

Closed bkmgit closed 1 year ago

bkmgit commented 1 year ago

Make a copy of the file.

gcapes commented 1 year ago

Thanks for the PR. This is a callout showing what not to do. So I'm afraid that changing the example to a better practice negates the point of the callout.

emcaulay commented 1 year ago

Now, I'm seeing that this PR was not accepted, but I do think it is a good revision and addresses the concern discussed in #1375. I think it seems a little cruel to set up a learner to destroy a file by mistake when we expect them to have the intact file for continuing the lesson.