Skip to content

feat: add not-starts/ends-with operators - #227

Merged
fdncred merged 3 commits into
nushell:mainfrom
Tyarel8:not-startends-with
Oct 13, 2025
Merged

feat: add not-starts/ends-with operators#227
fdncred merged 3 commits into
nushell:mainfrom
Tyarel8:not-startends-with

Conversation

@Tyarel8

@Tyarel8 Tyarel8 commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

New operators added in nushell/nushell#16852

@blindFS

blindFS commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

Would you please add new tests to this file?

@Tyarel8

Tyarel8 commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

Added. Please let me know if I did something wrong, I have no idea about tree-sitter.

fdncred pushed a commit to nushell/nushell that referenced this pull request Oct 12, 2025
<!--
Thank you for improving Nushell!
Please, read our contributing guide:
https://github.com/nushell/nushell/blob/main/CONTRIBUTING.md
-->

Since no one seems opposed to it
https://discord.com/channels/601130461678272522/615329862395101194/1426617232074608650

## Release notes summary - What our users need to know
<!--
This section will be included as part of our release notes. See the
contributing guide for more details.
Please include only details relevant for users here. Motivation and
technical details can be added above or below this section.

You may leave this section blank until your PR is finalized. Ask a core
team member if you need help filling this section.
-->
Added `not-starts-with` and `not-ends-with` operators to be on par with
other comparison operators,

## Tasks after submitting
<!-- Remove any tasks which aren't relevant for your PR, or add your own
-->
- [x] Add tests
- [x] Update the
[documentation](https://github.com/nushell/nushell.github.io).
nushell/nushell.github.io#2054
- [x] Update the tree-sitter grammar.
nushell/tree-sitter-nu#227
@blindFS

blindFS commented Oct 12, 2025

Copy link
Copy Markdown
Contributor

Added. Please let me know if I did something wrong, I have no idea about tree-sitter.

Looks good, but we also track the generated c/json code in this repo.
Here're some extra steps if not done yet:

  1. run tree-sitter generate
  2. run tree-sitter test to make sure everything works
  3. commit the change made in step 1

@Tyarel8

Tyarel8 commented Oct 12, 2025

Copy link
Copy Markdown
Contributor Author

I have also committed the generated tree-sitter and all tests pass

@fdncred
fdncred merged commit d9d6ad2 into nushell:main Oct 13, 2025
4 checks passed
@fdncred

fdncred commented Oct 13, 2025

Copy link
Copy Markdown
Contributor

Thanks

@Tyarel8
Tyarel8 deleted the not-startends-with branch October 13, 2025 19:12
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.

3 participants