Skip to content

Changes misleading example #525

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

Merged
merged 1 commit into from
Jul 2, 2022
Merged

Changes misleading example #525

merged 1 commit into from
Jul 2, 2022

Conversation

merelymyself
Copy link
Contributor

It seemed to suggest that $it was necessary in {|$somevar| $somevar.size > 10kb }.

It seemed to suggest that $it was necessary in {|$somevar| $somevar.size > 10kb }.
@fdncred
Copy link
Contributor

fdncred commented Jul 2, 2022

Right. You can name the variable whatever you want. Thanks.

@fdncred fdncred merged commit 931d724 into nushell:main Jul 2, 2022
ayax79 pushed a commit to ayax79/nushell.github.io that referenced this pull request Jun 26, 2024
* move "create release PR" script to `make_release/release-note/`

* move "since last release" script to `make_release/release-note/`

* add a script to list contributions for the release note

* rewrite `since_last_release` to use `list-merged-prs`

This commit also makes the script executable.

* update the `make_release/` readme

* add the commands to run to the release note template

* rename `since_last_release.nu` to `get-full-changelog`

* move the release note PR template to a standalone file

* add another section to inspect Nushell PRs to write changelogs

* fix the name of the command inside it's own doc
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