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

Expand append examples and add match example #1083

Merged
merged 5 commits into from
Sep 30, 2023
Merged

Expand append examples and add match example #1083

merged 5 commits into from
Sep 30, 2023

Conversation

savente93
Copy link
Contributor

As discussed in nushell/nushell#10486 Some more examples of the various ways to append things to lists/strings/tables would be appreciated. In addition, str append was a nice exmaple of how to use match and also pattern match on types.

This PR adds more examples on the appending in the appropriate sections (specifically more on using the ++ operator which was previously not mentioned).

str append is also added to the cookbook and expanded upon somewhat. In the linked issue "vectorisation" was also mentioned but I wasn't quite sure how/what part of the str append that refers to so I didn't mention that. If someone can explain that part to me and thinks it would be useful to also add, I'd be happy to do so.

Finally I think instructions on how to add new pages to the cookbook were outdated (at least as far as I could tell) so I updated that as well in accordance with how I did it. If this is incorrect, Let me know how it should be done instead and I'll happily change it up

Append exmaples now also include examples on how
to use ++ for example, which was a bit lacking.

For `match` the `str append` example is added
and expanded on
@savente93
Copy link
Contributor Author

@amtoine Would you mind reviewing? (you mentioned in the issue I could ping you)

Copy link
Collaborator

@fdncred fdncred left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Very nice improvements to our documentation. Thanks!

@fdncred
Copy link
Collaborator

fdncred commented Sep 30, 2023

I fixed a couple typos. Good work! Looks like there are a couple more typos left to fix.

Copy link
Member

@amtoine amtoine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

i like that a lot 🙏

i have a few minor remarks and, if you fix the CI for typos, i think we'll be able to land this soon 😊

book/working_with_strings.md Outdated Show resolved Hide resolved
cookbook/pattern_matching.md Outdated Show resolved Hide resolved
cookbook/pattern_matching.md Outdated Show resolved Hide resolved
@savente93
Copy link
Contributor Author

Thanks for the comments both! should be all good now 😊

Copy link
Member

@amtoine amtoine left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

thanks for the quick fixes and making our documentation better, much appreciated 🙏

i don't see anything more to change, let's land this 🥳
in case there is a typo or something off, we can fix them later 😌

@fdncred fdncred merged commit 6a57895 into nushell:main Sep 30, 2023
2 checks passed
@savente93 savente93 deleted the str-append branch September 30, 2023 13:59
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

3 participants