jumpstarter-driver-gpio: Add read method required by PowerInterface

[anobli]

This addresses the following issue:
TypeError: Can't instantiate abstract class PowerSwitch without an implementation for abstract method 'read'

This adds the read method which return 0 power because we can't get this kind of data from a gpio.

[coderabbitai]

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info

⚙️ Run configuration

Configuration used: Organization UI

Review profile: CHILL

Plan: Pro

Run ID: 358d5f8e-c27b-46bb-862b-c88c1045f012

📥 Commits

Reviewing files that changed from the base of the PR and between 7956f62 and 9370d0d.

📒 Files selected for processing (1)

• python/packages/jumpstarter-driver-gpiod/jumpstarter_driver_gpiod/driver.py

---

📝 Walkthrough

Walkthrough

A new read() method was added to the PowerSwitch class with a generator-based return type Generator[PowerReading, None, None]; the method body currently raises NotImplementedError and includes a yield to make it a generator. Generator and PowerReading were imported to support the signature.

Changes

Cohort / File(s)	Summary
GPIO Power Switch driver python/packages/jumpstarter-driver-gpiod/jumpstarter_driver_gpiod/driver.py	Added a stub read() method on PowerSwitch typed as Generator[PowerReading, None, None] that raises NotImplementedError and contains a yield (to ensure generator form). Added imports for Generator and PowerReading supporting the new signature.

Estimated code review effort

🎯 2 (Simple) | ⏱️ ~8 minutes

Poem

🐇 I added a little stub today,
A generator that won’t yet play,
It yields a promise, not the light—
Soon it’ll measure, soon it’ll write. ✨

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name	Status	Explanation	Resolution
Docstring Coverage	⚠️ Warning	Docstring coverage is 50.00% which is insufficient. The required threshold is 80.00%.	Write docstrings for the functions missing them to satisfy the coverage threshold.

✅ Passed checks (2 passed)

Check name	Status	Explanation
Title check	✅ Passed	The title clearly and specifically describes the main change: adding a read method to satisfy the PowerInterface requirement for the GPIO driver.
Description check	✅ Passed	The description is directly related to the changeset, explaining the problem (TypeError from missing abstract method) and the solution (adding the read method).

_{✏️ Tip: You can configure your own custom pre-merge checks in the settings.}

✨ Finishing Touches

🧪 Generate unit tests (beta)

• Create PR with unit tests

---

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

• X
• Mastodon
• Reddit
• LinkedIn

_{Comment @coderabbitai help to get the list of available commands and usage tips.}

[mangelajo]

Thank you very much for the bugfix.

[mangelajo]

Thank you so much

[jumpstarter-backport-bot]

Successfully created backport PR for release-0.8:

• [Backport release-0.8] jumpstarter-driver-gpio: Add read method required by PowerInterface #623