Skip to content

Conversation

NotTheDr01ds
Copy link
Contributor

@NotTheDr01ds NotTheDr01ds commented Aug 2, 2024

Continuing the general clean-up of this chapter that I started when adding the "multi-dot" info.

  • General grammar and wording tweaks - In general, tried to remove prepositions, especially "we" (but not all), and focus more on what Nushell does. This involves the use of more passive voice, but I hope it reads more professionally in most cases.
  • Linked to other sections to reduce redundancy
  • Removed the wording about Nushell passing globs "as is" to commands, since this only applies to internals, not externals. Also it was quite out-of-date, since all applicable filesystem commands now accept ...rest
  • Added raw strings as a valid glob-escaping method
  • Clarified the explanation/example of why backticks don't escape globs
  • Likewise, the "string to glob" conversion was outdated due to the ...rest changes
  • Removed the examples from the cp, mv, etc. commands since they are already linked to the command help itself, and these already include better and more complete examples.
  • Other improvements

@NotTheDr01ds
Copy link
Contributor Author

@fdncred You're right - That's what I say in the comment, but then the code example isn't what I thought I put there. Let me see what went wrong, but I definitely will push a fix for that in a moment.

@fdncred fdncred marked this pull request as draft August 2, 2024 16:45
@NotTheDr01ds NotTheDr01ds marked this pull request as ready for review August 2, 2024 19:13
@fdncred fdncred merged commit 44e8fec into nushell:main Aug 3, 2024
2 checks passed
@fdncred
Copy link
Contributor

fdncred commented Aug 3, 2024

Thanks

@NotTheDr01ds NotTheDr01ds deleted the more-moving-around branch August 3, 2024 15:36
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.

2 participants