Skip to content

labgrid/resource/remote: add missing sigrok channel_group parameter#1562

Merged
Bastian-Krause merged 1 commit intolabgrid-project:masterfrom
pslldq:fixsigrok
Feb 4, 2025
Merged

labgrid/resource/remote: add missing sigrok channel_group parameter#1562
Bastian-Krause merged 1 commit intolabgrid-project:masterfrom
pslldq:fixsigrok

Conversation

@pslldq
Copy link
Contributor

@pslldq pslldq commented Jan 7, 2025

Description

Add the missing sigrok channel_group parameter to the Network variants of the sigrok resources.

Checklist

  • PR has been tested

Fixes: #1536

Add the missing sigrok channel_group parameter to the Network variants
of the sigrok resources.

Fixes: d6c5973 ("driver/sigrokdriver: add channel-group parameter")
Signed-off-by: Sven Püschel <s.pueschel@pengutronix.de>
@codecov
Copy link

codecov bot commented Jan 7, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 56.2%. Comparing base (587a872) to head (5b3ccd0).
Report is 87 commits behind head on master.

✅ All tests successful. No failed tests found.

Additional details and impacted files
@@          Coverage Diff           @@
##           master   #1562   +/-   ##
======================================
  Coverage    56.2%   56.2%           
======================================
  Files         170     170           
  Lines       13247   13249    +2     
======================================
+ Hits         7449    7451    +2     
  Misses       5798    5798           
Flag Coverage Δ
3.10 56.5% <100.0%> (+<0.1%) ⬆️
3.11 56.5% <100.0%> (+<0.1%) ⬆️
3.12 56.5% <100.0%> (+<0.1%) ⬆️
3.13 56.5% <100.0%> (+<0.1%) ⬆️
3.9 56.5% <100.0%> (+<0.1%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

@Bastian-Krause Bastian-Krause merged commit fcd2698 into labgrid-project:master Feb 4, 2025
10 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants