Skip to content

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

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

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

Conversation

@Tyarel8
Copy link
Contributor

@Tyarel8 Tyarel8 commented Oct 12, 2025

New operators added in nushell/nushell#16852

@blindFS
Copy link
Contributor

blindFS commented Oct 12, 2025

Would you please add new tests to this file?

@Tyarel8
Copy link
Contributor Author

Tyarel8 commented Oct 12, 2025

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
Copy link
Contributor

blindFS commented Oct 12, 2025

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
Copy link
Contributor Author

Tyarel8 commented Oct 12, 2025

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
Copy link
Contributor

fdncred commented Oct 13, 2025

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