Skip to content
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

Fix: Expect ShellCommandResult from executing ShellCommandRequest #74

Merged
merged 1 commit into from
Feb 8, 2022

Conversation

inez
Copy link
Contributor

@inez inez commented Feb 7, 2022

No description provided.

@inez inez requested a review from Malinskiy as a code owner February 7, 2022 00:07
@codecov
Copy link

codecov bot commented Feb 7, 2022

Codecov Report

Merging #74 (b0d9ec3) into master (e83efc8) will not change coverage.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff            @@
##             master      #74   +/-   ##
=========================================
  Coverage     80.75%   80.75%           
  Complexity      711      711           
=========================================
  Files           141      141           
  Lines          3087     3087           
  Branches        497      497           
=========================================
  Hits           2493     2493           
  Misses          341      341           
  Partials        253      253           
Flag Coverage Δ
integration 63.33% <ø> (ø)
unit 71.52% <ø> (+0.03%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.


Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update e83efc8...b0d9ec3. Read the comment docs.

@inez
Copy link
Contributor Author

inez commented Feb 8, 2022

@Malinskiy "Only those with write access to this repository can merge pull requests." - Seems that you will have to be the one to merge it.

@Malinskiy Malinskiy merged commit 7291ad7 into Malinskiy:master Feb 8, 2022
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.

None yet

2 participants