-
Notifications
You must be signed in to change notification settings - Fork 760
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’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
chore: fix ShellCheck errors in tests #1433
Conversation
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.
Pressed the wrong button 😫
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.
See comments for requested changes.
I will approve & merge afterwards, cheers!
I know this is a slow process, sorry @hyperupcall your help is appreciated 🙏
8dc86c9
to
aef5eb3
Compare
aef5eb3
to
f4b4d11
Compare
Rebased over |
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.
Looks great, thanks for the changes!
On to the next! |
woot! |
Summary
This PR is the first of a multi-part series that fixes various issues with the tests. I broke this change up as to be easier to review, and it seemed similar to #1349, in which different PRs were preferred over a single larger one.
Here is a description of each commit/part:
$?
withrun
(chore: Fix ShellCheck errors in tests #1451)At the end of the commit series, some lints were disabled:
test/reshim_command.bats
since I wanted to be sure not to break any tests and the conversion tofind
isn't as trivial as the other fixes - it can be fixed later.Partially addresses #1396