Skip to content

Conversation

@afflux
Copy link
Owner

@afflux afflux commented Apr 11, 2023

No description provided.

@afflux afflux force-pushed the feature/multicast-eventgroups branch from 5679954 to 93c95d6 Compare April 11, 2023 14:05
@codecov
Copy link

codecov bot commented Apr 11, 2023

Codecov Report

Attention: Patch coverage is 84.50704% with 11 lines in your changes missing coverage. Please review.

Project coverage is 95.69%. Comparing base (ab378e9) to head (fcc952c).
Report is 2 commits behind head on master.

Files with missing lines Patch % Lines
src/someip/sd.py 80.00% 5 Missing and 1 partial ⚠️
src/someip/service.py 87.80% 4 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #12      +/-   ##
==========================================
- Coverage   96.55%   95.69%   -0.87%     
==========================================
  Files           6        6              
  Lines        1308     1348      +40     
  Branches      210      221      +11     
==========================================
+ Hits         1263     1290      +27     
- Misses         35       46      +11     
- Partials       10       12       +2     

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@afflux afflux force-pushed the feature/multicast-eventgroups branch from 93c95d6 to fcc952c Compare August 31, 2023 15:51
@werux
Copy link

werux commented May 8, 2024

Hi @afflux, is there any reason why this is still in draft mode?

I may be able to help test it and review.

@afflux
Copy link
Owner Author

afflux commented May 8, 2024

Hi @werux, I haven't yet had a chance to verify if it actually works against real implementations. Feel free to give it a go and let me know how it went.

@werux
Copy link

werux commented May 30, 2024

Hi @afflux,

I haven't had the chance to test this, but I submitted a PR to add support for SOME/IP TP (#19), can you please take a look?

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.

3 participants