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 def-env
and export def-env
#10999
Conversation
- remove the files with ```nushell ls **/*.rs | find --regex def_env | each { rm --verbose $in.name }` ``` - run `:g/DefEnv/d` in `crates/nu-cmd-lang/src/default_context.rs` - run `:g/def_env/d` in `crates/nu-cmd-lang/src/core_commands/mod.rs`
5060475
to
1fcbeb7
Compare
Hi @amtoine , would you please fix the conflicts? I believe it's conflict with |
this should solve the merge conflict from nushell#10999
yup, should be better now @WindSoilder 馃槈 |
I'm sorry that I didn't review it yesterday, it seems that we still have comflicts...Can you please resolve it again? |
this should solve the merge conflict in nushell#10999.
no worries @WindSoilder, should be good to go again 馃槈 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Sure! Let's go, thanks for your work through this
follow-up to - nushell#10715 > **Important** > wait for between 0.87 and 0.88 to land this # Description it's time for removal again 馃構 this PR removes `def-env` and `export def-env` in favor of `def --env` # User-Facing Changes `def-env` and `export def-env` will not be found anymore. # Tests + Formatting # After Submitting
follow-up to - nushell#10715 > **Important** > wait for between 0.87 and 0.88 to land this # Description it's time for removal again 馃構 this PR removes `def-env` and `export def-env` in favor of `def --env` # User-Facing Changes `def-env` and `export def-env` will not be found anymore. # Tests + Formatting # After Submitting
follow-up to
def-env
andexport def-env
聽#10715Description
it's time for removal again 馃構
this PR removes
def-env
andexport def-env
in favor ofdef --env
User-Facing Changes
def-env
andexport def-env
will not be found anymore.Tests + Formatting
After Submitting