Skip to content

Use a case-insensitive compare to set the options of SelectionBase#219

Merged
kbabioch merged 1 commit intoBouni:mainfrom
Guzz-T:try/case-insensitive-options
Jan 6, 2026
Merged

Use a case-insensitive compare to set the options of SelectionBase#219
kbabioch merged 1 commit intoBouni:mainfrom
Guzz-T:try/case-insensitive-options

Conversation

@Guzz-T
Copy link
Contributor

@Guzz-T Guzz-T commented Jan 5, 2026

It is not necessary to pay attention to upper/lower case here.
The new implementation is also more user-friendly.
Additional "-" now matches "_".

It is not necessary to pay attention to upper/lower case here. The new implementation is also more user-friendly.
Additional "-" now matches "_".
@github-actions
Copy link

github-actions bot commented Jan 5, 2026

Coverage

Coverage Report
FileStmtsMissCoverMissing
luxtronik
   __init__.py2361394%89–102, 251–252, 257
   __main__.py23230%3–52
   datatypes.py395199%121
   discover.py58198%62
   parameters.py24196%1223
luxtronik/scripts
   __init__.py16160%3–28
   discover_shi.py60600%8–98
   dump_changes.py43430%7–85
   dump_luxtronik.py26260%6–52
   dump_shi.py19190%7–40
   performance_shi.py63630%7–89
   watch_shi.py38380%7–85
luxtronik/shi
   contiguous.py135299%261–262
TOTAL219530686% 

Tests Skipped Failures Errors Time
363 0 💤 0 ❌ 0 🔥 13.943s ⏱️

@kbabioch kbabioch merged commit aa7667b into Bouni:main Jan 6, 2026
4 checks passed
@Guzz-T Guzz-T deleted the try/case-insensitive-options branch January 6, 2026 20:59
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