Nukesor / pueue

:stars: Manage your shell commands.
Apache License 2.0
5.04k stars 134 forks source link

Use GetCurrentProcess instead of OpenProcess #570

Closed MolotovCherry closed 1 month ago

MolotovCherry commented 1 month ago

Just a minor code change improvement to simplify one call. Replaced OpenProcess with GetCurrentProcess since this is what we're doing anyways

Checklist

Please make sure the PR adheres to this project's standards:

codecov[bot] commented 1 month ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Project coverage is 78.24%. Comparing base (92af798) to head (a3c3caf). Report is 3 commits behind head on main.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #570 +/- ## ======================================= Coverage 78.24% 78.24% ======================================= Files 74 74 Lines 5898 5898 ======================================= Hits 4615 4615 Misses 1283 1283 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

github-actions[bot] commented 1 month ago

Test Results

  3 files  ±0   19 suites  ±0   3m 5s :stopwatch: -1s 161 tests ±0  161 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0  340 runs  ±0  340 :white_check_mark: ±0  0 :zzz: ±0  0 :x: ±0 

Results for commit a3c3caf7. ± Comparison against base commit 92af7989.

Nukesor commented 1 month ago

Nice