dclause/hermes-five

Use `Into<PinIdOrName>` in all device/expanders constructors

Open

#21 opened on Jun 15, 2025

View on GitHub
 (0 comments) (0 reactions) (0 assignees)Rust (1 fork)auto 404
code qualityfeature requestgood first issue

Repository metrics

Stars
 (37 stars)
PR merge metrics
 (PR metrics pending)

Description

Motivation: A PinIdOrName enum was introduced to simplify the use of digital and analog pins (e.g., "D11", "A11"). To make the API more ergonomic and consistent, this enum should be accepted in all constructors that require a pin, including device drivers and expanders.

API Change: No breaking change. This is a backward-compatible improvement via the Into<PinIdOrName> trait, enabling seamless use of both u8 and String literals.

Contributor guide

Research direction
Identify all device and expander constructors that accept a pin parameter and modify them to accept `Into<PinIdOrName>` instead of concrete types.
Tech stack
rust
Domain
embedded iot
Issue type
Refactor
DifficultyEstimated implementation difficulty for a new contributor, from 1 for very small changes to 5 for expert-level work.
1
Estimated timeA rough time range for an experienced contributor to investigate, implement, test, and prepare a pull request.
1-3 hours
Activity statusHow available the issue appears right now: fresh, active, stale, blocked, or waiting on maintainer input.
Fresh
ClarityHow clearly the issue explains the expected change, acceptance criteria, and next step.
Clear
Prerequisites
RustGit
Newbie friendlinessA 1-100 score estimating how approachable this issue is for first-time contributors.
80