Skip to content

Comments

Fix disown#643

Merged
KCarretto merged 9 commits intomainfrom
624-download-and-execute-tome-doesnt-cleanup-tmp-file
Feb 21, 2024
Merged

Fix disown#643
KCarretto merged 9 commits intomainfrom
624-download-and-execute-tome-doesnt-cleanup-tmp-file

Conversation

@hulto
Copy link
Collaborator

@hulto hulto commented Feb 21, 2024

What type of PR is this?

/kind bug

What this PR does / why we need it:

Fixes linux disown not working.

@hulto hulto linked an issue Feb 21, 2024 that may be closed by this pull request
@Cictrone
Copy link
Collaborator

windows failing!

Copy link
Collaborator

@Cictrone Cictrone left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

pls fix

@codecov
Copy link

codecov bot commented Feb 21, 2024

Codecov Report

Attention: 2 lines in your changes are missing coverage. Please review.

Comparison is base (2868dbc) 72.06% compared to head (1ae4d8d) 72.06%.

Files Patch % Lines
implants/lib/eldritch/src/sys/exec_impl.rs 88.88% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #643   +/-   ##
=======================================
  Coverage   72.06%   72.06%           
=======================================
  Files         174      174           
  Lines       11964    11968    +4     
=======================================
+ Hits         8622     8625    +3     
- Misses       3128     3129    +1     
  Partials      214      214           

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Cictrone
Cictrone previously approved these changes Feb 21, 2024
@KCarretto KCarretto dismissed Cictrone’s stale review February 21, 2024 05:31

The merge-base changed after approval.

@KCarretto KCarretto merged commit 322b54a into main Feb 21, 2024
@KCarretto KCarretto deleted the 624-download-and-execute-tome-doesnt-cleanup-tmp-file branch February 21, 2024 16:55
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.

Download and execute tome doesn't cleanup tmp file

3 participants