dclause/hermes-five

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

Aberta

#21 aberto em 15 de jun. de 2025

Ver no GitHub
 (0 comentário) (0 reação) (0 responsável)Rust (1 fork)auto 404
code qualityfeature requestgood first issue

Métricas do repositório

Stars
 (37 estrelas)
Métricas de merge de PR
 (Métricas PR pendentes)

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.

Guia do colaborador

Direção de pesquisa
Identifique todos os construtores de dispositivos e expansores que aceitam um parâmetro de pino e modifique os para aceitar `Into<PinIdOrName>` em vez de tipos concretos.
Pilha de tecnologia
rust
Domain
embedded iot
Tipo Issue
Refatoração
DifficultyDificuldade de implementação estimada para um novo contribuidor, de 1 para alterações muito pequenas a 5 para trabalho de nível especializado.
1
Tempo estimadoUm intervalo de tempo aproximado para um colaborador experiente investigar, implementar, testar e preparar um pull request.
1-3 horas
Status da atividadeQuão disponível o issue aparece agora: novo, ativo, obsoleto, bloqueado ou aguardando entrada do mantenedor.
Recente
ClarityCom que clareza o issue explica a mudança esperada, os critérios de aceitação e a próxima etapa.
Claro
Prerequisites
RustGit
Simpatia para novatosUma pontuação de 1 a 100 que estima o quão acessível este issue é para colaboradores iniciantes.
80