Skip to content

Comments

Fix implants windows release workflow#582

Merged
Milkshak3s merged 2 commits intomainfrom
Milkshak3s-implants-windows-release-fix
Feb 13, 2024
Merged

Fix implants windows release workflow#582
Milkshak3s merged 2 commits intomainfrom
Milkshak3s-implants-windows-release-fix

Conversation

@Milkshak3s
Copy link
Collaborator

What type of PR is this?

/kind devops
/kind bug

What this PR does / why we need it:

This should(?) might(?) fix windows builds for implants during releases.
Tested in #581 without uploading, so I don't know if it produces the correct bin, but it does finish the build without errors.

Which issue(s) this PR fixes:

Fixes #

@Milkshak3s Milkshak3s added bug Something isn't working dev-ops labels Feb 13, 2024
@codecov
Copy link

codecov bot commented Feb 13, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Comparison is base (ff93177) 67.68% compared to head (7ce36a3) 71.83%.

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #582      +/-   ##
==========================================
+ Coverage   67.68%   71.83%   +4.15%     
==========================================
  Files         142      142              
  Lines        9382     9955     +573     
==========================================
+ Hits         6350     7151     +801     
+ Misses       2866     2638     -228     
  Partials      166      166              

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

RUSTFLAGS: "-C target-feature=+crt-static"
- name: 🔨 Build Imix (DLL)
run: cargo build --lib --release
run: cargo build --lib --release --verbose --target=x86_64-pc-windows-gnu
Copy link
Collaborator

Choose a reason for hiding this comment

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

Do we want to leave verbose in?

Copy link
Collaborator

Choose a reason for hiding this comment

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

Yes :)

Copy link
Collaborator

@hulto hulto left a comment

Choose a reason for hiding this comment

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

Looks good

@Milkshak3s Milkshak3s merged commit 3ea7d14 into main Feb 13, 2024
@Milkshak3s Milkshak3s deleted the Milkshak3s-implants-windows-release-fix branch February 13, 2024 02:31
nullmonk pushed a commit that referenced this pull request Feb 14, 2024
Co-authored-by: Hulto <7121375+hulto@users.noreply.github.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working dev-ops

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants