Skip to content

add benchmark to deprecated commands#8046

Merged
fdncred merged 1 commit intonushell:mainfrom
fdncred:deprecate_benchmark
Feb 11, 2023
Merged

add benchmark to deprecated commands#8046
fdncred merged 1 commit intonushell:mainfrom
fdncred:deprecate_benchmark

Conversation

@fdncred
Copy link
Copy Markdown
Contributor

@fdncred fdncred commented Feb 11, 2023

Description

This PR adds a deprecated message to the benchmark command.

> benchmark
Error: nu::shell::deprecated_command (link)

  × Deprecated command benchmark
   ╭─[entry #1:1:1]
 1 │ benchmark
   · ────┬────
   ·     ╰── 'benchmark' is deprecated. Please use 'timeit' instead.
   ╰────

User-Facing Changes

Tests + Formatting

Don't forget to add tests that cover your changes.

Make sure you've run and fixed any issues with these commands:

  • cargo fmt --all -- --check to check standard code formatting (cargo fmt --all applies these changes)
  • cargo clippy --workspace -- -D warnings -D clippy::unwrap_used -A clippy::needless_collect to check that you're using the standard code style
  • cargo test --workspace to check that all tests pass

After Submitting

If your PR had any user-facing changes, update the documentation after the PR is merged, if necessary. This will help us keep the docs up to date.

@codecov-commenter
Copy link
Copy Markdown

codecov-commenter commented Feb 11, 2023

Codecov Report

Base: 55.28% // Head: 54.90% // Decreases project coverage by -0.38% ⚠️

Coverage data is based on head (2b006d6) compared to base (0780300).
Patch coverage: 100.00% of modified lines in pull request are covered.

📣 This organization is not using Codecov’s GitHub App Integration. We recommend you install it so Codecov can continue to function properly for your repositories. Learn more

Additional details and impacted files
@@            Coverage Diff             @@
##             main    #8046      +/-   ##
==========================================
- Coverage   55.28%   54.90%   -0.38%     
==========================================
  Files         605      605              
  Lines       98847    98848       +1     
==========================================
- Hits        54647    54274     -373     
- Misses      44200    44574     +374     
Impacted Files Coverage Δ
...s/nu-command/src/deprecated/deprecated_commands.rs 100.00% <100.00%> (ø)
crates/nu-color-config/src/nu_style.rs 9.31% <0.00%> (-70.92%) ⬇️
crates/nu-color-config/src/style_computer.rs 80.76% <0.00%> (-0.55%) ⬇️
crates/nu-command/src/filesystem/touch.rs 63.73% <0.00%> (-0.52%) ⬇️
crates/nu-command/src/charting/hashable_value.rs 89.25% <0.00%> (+0.46%) ⬆️

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

☔ View full report at Codecov.
📢 Do you have feedback about the report comment? Let us know in this issue.

@fdncred fdncred added this pull request to the merge queue Feb 11, 2023
Merged via the queue into nushell:main with commit 9777d75 Feb 11, 2023
@fdncred fdncred deleted the deprecate_benchmark branch February 11, 2023 21:40
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.

2 participants