Skip to content

cmds/core/which: fix default usage#2515

Merged
probot-auto-merge[bot] merged 2 commits into
u-root:mainfrom
binjip978:which-default
Oct 10, 2022
Merged

cmds/core/which: fix default usage#2515
probot-auto-merge[bot] merged 2 commits into
u-root:mainfrom
binjip978:which-default

Conversation

@binjip978
Copy link
Copy Markdown
Contributor

which cmd1 cmd2 should not return after first find, it should continiue with next command

Signed-off-by: Siarhiej Siemianchuk pdp.eleven11@gmail.com

`which cmd1 cmd2` should not return after first find, it
should continiue with next command

Signed-off-by: Siarhiej Siemianchuk <pdp.eleven11@gmail.com>
@codecov
Copy link
Copy Markdown

codecov Bot commented Oct 9, 2022

Codecov Report

Base: 73.74% // Head: 73.74% // No change to project coverage 👍

Coverage data is based on head (3f25bf7) compared to base (725e1eb).
Patch coverage: 100.00% of modified lines in pull request are covered.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #2515   +/-   ##
=======================================
  Coverage   73.74%   73.74%           
=======================================
  Files         403      403           
  Lines       40928    40928           
=======================================
  Hits        30183    30183           
  Misses      10745    10745           
Impacted Files Coverage Δ
cmds/core/which/which.go 52.00% <100.00%> (+4.00%) ⬆️
cmds/core/mv/mv.go 77.27% <0.00%> (-2.28%) ⬇️

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.

@rminnich rminnich added automerge Applying this label auto-merges the PR when ready Awaiting author Waiting for new changes or feedback for author. labels Oct 10, 2022
@probot-auto-merge probot-auto-merge Bot merged commit afc634c into u-root:main Oct 10, 2022
@binjip978 binjip978 deleted the which-default branch November 15, 2023 18:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

automerge Applying this label auto-merges the PR when ready Awaiting author Waiting for new changes or feedback for author.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants