tooling: Enable ruff S110 to detect try-except-pass.

[nedseb]

Closes #384

Problem

Silent try/except: pass patterns hide errors. The github-code-quality bot detected such patterns in PR #382 (deploy_usb.py), but ruff did not catch them automatically — meaning future occurrences would also slip through review.

Fix

Enable the S110 rule from flake8-bandit (already part of ruff) which detects try-except-pass:

"S110",   # flake8-bandit: try-except-pass (no silent exception suppression)

Only S110 is enabled (not the full S group), because the other bandit rules (S101 assert, S102 exec, S603/S607 subprocess) produce too many false positives in our test/script code.

Existing violations fixed

Two try-except: pass found in the codebase:

1. lib/lis2mdl/device.py — unnecessary wrapper around sleep_ms

# Before
try:
    sleep_ms(10)  # Small delay for reset to complete
except Exception:
    pass

# After
sleep_ms(10)  # Small delay for reset to complete

sleep_ms() cannot raise — the try/except was a leftover from older code.

2. tests/report_plugin.py — intentional best-effort fallback

Kept with # noqa: S110 and a clarifying comment. The report plugin must never fail the test session if metadata collection fails.

Test plan

• ruff check passes
• 349 mock tests pass

[Copilot]

Pull request overview

This PR tightens repository linting by enabling Ruff’s flake8-bandit rule S110, so silent try/except: pass patterns are flagged during CI and don’t slip through review.

Changes:

• Enable Ruff rule S110 to detect try-except-pass.
• Remove an unnecessary try/except: pass around sleep_ms() in the LIS2MDL driver.
• Keep an intentional best-effort try/except: pass in the pytest report plugin, documented and suppressed with # noqa: S110.

Reviewed changes

Copilot reviewed 3 out of 3 changed files in this pull request and generated 1 comment.

File	Description
pyproject.toml	Adds S110 to Ruff’s lint select list to enforce detection of silent exception suppression.
lib/lis2mdl/lis2mdl/device.py	Removes a redundant try/except: pass around a fixed sleep_ms(10) delay during reset.
tests/report_plugin.py	Documents and suppresses an intentional best-effort try/except: pass used to avoid failing the test session on metadata collection issues.

---

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

[Copilot]

The except Exception: ... pass block requires # noqa: S110 and adds a redundant pass before the unconditional return {}. Consider returning {} directly from the except block (and dropping the noqa) to keep the best-effort behavior while avoiding a suppressed try/except/pass pattern.

Suggested change

	except Exception: # noqa: S110
	# Best-effort: report metadata is optional, never fail the test session.
	pass
	except Exception:
	# Best-effort: report metadata is optional, never fail the test session.
	return {}

[nedseb]

Fixed: replaced pass with return {} directly in the except block. The # noqa: S110 is no longer needed since there is no try-except-pass anymore.

[semantic-release-updater]

🎉 This PR is included in version 0.16.4 🎉

The release is available on:

• v0.16.4
• GitHub release

Your semantic-release bot 📦🚀