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(fig_desktop): exit code not propagated by main #409

Merged
merged 1 commit into from
Jan 10, 2025

Conversation

grant0417
Copy link
Member

@grant0417 grant0417 commented Jan 10, 2025

This fixes an issues that was accidently introduced in #394. To prevent this again I have marked the functions in the file as #[must_use].

By submitting this pull request, I confirm that you can use, modify, copy, and redistribute this contribution, under the terms of your choice.

@grant0417 grant0417 requested a review from a team as a code owner January 10, 2025 20:04
@grant0417 grant0417 force-pushed the fix-exit-code branch 2 times, most recently from 9b753d2 to 5931abd Compare January 10, 2025 20:06
This fixes an issues that was accidently introduced in 524eec6. To prevent this again I have marked the functions in the file as `#[must_use]`.
@grant0417 grant0417 changed the title fix: fig_desktop exit code return fix(fig_desktop): exit code not propogated by main Jan 10, 2025
@grant0417 grant0417 changed the title fix(fig_desktop): exit code not propogated by main fix(fig_desktop): exit code not propagated by main Jan 10, 2025
@codecov-commenter
Copy link

codecov-commenter commented Jan 10, 2025

Codecov Report

Attention: Patch coverage is 0% with 6 lines in your changes missing coverage. Please review.

Project coverage is 12.43%. Comparing base (524eec6) to head (a83f257).
Report is 2 commits behind head on main.

Files with missing lines Patch % Lines
crates/fig_desktop/src/main.rs 0.00% 6 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #409   +/-   ##
=======================================
  Coverage   12.43%   12.43%           
=======================================
  Files        2233     2233           
  Lines      189905   189902    -3     
  Branches   174493   174490    -3     
=======================================
  Hits        23608    23608           
+ Misses     165116   165113    -3     
  Partials     1181     1181           

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

@grant0417 grant0417 merged commit 921c861 into aws:main Jan 10, 2025
10 checks passed
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.

2 participants