Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Streamline message names and preparations for multiple followed tasks #617

Merged
merged 2 commits into from
Feb 21, 2025

Conversation

Nukesor
Copy link
Owner

@Nukesor Nukesor commented Feb 20, 2025

Streamlines all message names and adjusts the Request::Stream and Response::Stream for multiple tasks.

See #614

Copy link

codecov bot commented Feb 20, 2025

Codecov Report

Attention: Patch coverage is 74.50980% with 26 lines in your changes missing coverage. Please review.

Project coverage is 80.28%. Comparing base (02a6d13) to head (36773bb).
Report is 3 commits behind head on main.

Files with missing lines Patch % Lines
pueue/src/daemon/network/message_handler/log.rs 66.66% 10 Missing ⚠️
pueue/src/client/commands/parallel.rs 0.00% 2 Missing ⚠️
pueue/src/daemon/network/message_handler/group.rs 60.00% 2 Missing ⚠️
pueue/src/client/commands/clean.rs 0.00% 1 Missing ⚠️
pueue/src/client/commands/group.rs 75.00% 1 Missing ⚠️
pueue/src/client/commands/kill.rs 0.00% 1 Missing ⚠️
pueue/src/client/commands/pause.rs 0.00% 1 Missing ⚠️
pueue/src/client/commands/reset.rs 0.00% 1 Missing ⚠️
pueue/src/client/commands/send.rs 0.00% 1 Missing ⚠️
pueue/src/client/commands/shutdown.rs 0.00% 1 Missing ⚠️
... and 5 more
Additional details and impacted files
@@            Coverage Diff             @@
##             main     #617      +/-   ##
==========================================
- Coverage   80.30%   80.28%   -0.02%     
==========================================
  Files          89       89              
  Lines        6381     6396      +15     
==========================================
+ Hits         5124     5135      +11     
- Misses       1257     1261       +4     

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

@Nukesor Nukesor merged commit 8587e79 into main Feb 21, 2025
18 of 19 checks passed
@Nukesor Nukesor deleted the messages branch February 21, 2025 02:17
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant