Skip to content

Commit

Permalink
Lazy record deprecation (#12656)
Browse files Browse the repository at this point in the history
# Description
In this week's nushell meeting, we decided to go ahead with #12622 and
remove lazy records in 0.94.0. For 0.93.0, we will only deprecate `lazy
make`, and so this PR makes `lazy make` print a deprecation warning.

# User-Facing Changes
None, besides the deprecation warning.

# After Submitting
Remove lazy records.
  • Loading branch information
IanManske committed Apr 24, 2024
1 parent 25cbcb5 commit 530162b
Showing 1 changed file with 12 additions and 0 deletions.
12 changes: 12 additions & 0 deletions crates/nu-cmd-lang/src/core_commands/lazy_make.rs
Original file line number Diff line number Diff line change
Expand Up @@ -54,6 +54,18 @@ impl Command for LazyMake {
call: &Call,
_input: PipelineData,
) -> Result<PipelineData, ShellError> {
nu_protocol::report_error_new(
engine_state,
&ShellError::GenericError {
error: "Deprecated command".into(),
msg: "warning: lazy records and the `lazy make` command will be removed in 0.94.0"
.into(),
span: Some(call.head),
help: None,
inner: vec![],
},
);

let span = call.head;
let columns: Vec<Spanned<String>> = call
.get_flag(engine_state, stack, "columns")?
Expand Down

0 comments on commit 530162b

Please sign in to comment.