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鈥檒l occasionally send you account related emails.

Already on GitHub? Sign in to your account

remove --not from glob #10839

Merged
merged 1 commit into from
Nov 16, 2023
Merged

remove --not from glob #10839

merged 1 commit into from
Nov 16, 2023

Conversation

amtoine
Copy link
Member

@amtoine amtoine commented Oct 25, 2023

follow-up to

Important
wait for between 0.87 and 0.88 to land this

Description

after deprecation comes removal... this PR removes glob --not in favor of glob --exclude.

User-Facing Changes

glob --not will stop working.

Tests + Formatting

After Submitting

i didn't find any use of glob --not in the nu_scripts so no update required there 馃憤

@amtoine amtoine added pr:breaking-change This PR implies a change affecting users and has to be noted in the release notes deprecation Related to the deprecation of commands labels Oct 25, 2023
@sholderbach sholderbach added removal-after-deprecation The component has already been sunset with `deprecation` and is now up for final removal and removed deprecation Related to the deprecation of commands labels Nov 8, 2023
@amtoine amtoine marked this pull request as ready for review November 16, 2023 17:46
Copy link
Collaborator

@WindSoilder WindSoilder left a comment

Choose a reason for hiding this comment

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

Thanks! LGTM

@WindSoilder WindSoilder merged commit ab59dab into nushell:main Nov 16, 2023
19 checks passed
@amtoine amtoine deleted the remove-glob-not branch November 17, 2023 13:42
melMass added a commit to amtoine/nu-git-manager that referenced this pull request Nov 17, 2023
related to
- nushell/nushell#10827
- nushell/nushell#10839

## description
`glob --not` has been deprecated in
nushell/nushell#10827 and will be removed in
nushell/nushell#10839 before 0.88.0 in favor of
`glob --exclude`.

this PR performs the change in `nu-git-manager`.

Co-authored-by: Mel Massadian <melmassadian@gmail.com>
hardfau1t pushed a commit to hardfau1t/nushell that referenced this pull request Dec 14, 2023
follow-up to
- nushell#10827

> **Important**  
> wait for between 0.87 and 0.88 to land this

# Description
after deprecation comes removal... this PR removes `glob --not` in favor
of `glob --exclude`.

# User-Facing Changes
`glob --not` will stop working.

# Tests + Formatting

# After Submitting
i didn't find any use of `glob --not` in the `nu_scripts` so no update
required there 馃憤
dmatos2012 pushed a commit to dmatos2012/nushell that referenced this pull request Feb 20, 2024
follow-up to
- nushell#10827

> **Important**  
> wait for between 0.87 and 0.88 to land this

# Description
after deprecation comes removal... this PR removes `glob --not` in favor
of `glob --exclude`.

# User-Facing Changes
`glob --not` will stop working.

# Tests + Formatting

# After Submitting
i didn't find any use of `glob --not` in the `nu_scripts` so no update
required there 馃憤
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
pr:breaking-change This PR implies a change affecting users and has to be noted in the release notes removal-after-deprecation The component has already been sunset with `deprecation` and is now up for final removal wait-until-after-nushell-release
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants