Skip to content
This repository has been archived by the owner on Sep 20, 2023. It is now read-only.

gpioreg: Remove ByNumber() #153

Merged
merged 1 commit into from
Aug 21, 2017
Merged

gpioreg: Remove ByNumber() #153

merged 1 commit into from
Aug 21, 2017

Conversation

maruel
Copy link
Contributor

@maruel maruel commented Aug 20, 2017

It is still possible to get a pin by its logical number using the string
representation, but the function is not exported anymore.

This simplifies the interface, one less way to do the same thing.

It is still possible to get a pin by its logical number using the string
representation, but the function is not exported anymore.

This simplifies the interface, one less way to do the same thing.
@maruel maruel requested a review from mattetti August 20, 2017 22:49
@codecov-io
Copy link

Codecov Report

Merging #153 into master will increase coverage by <.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #153      +/-   ##
==========================================
+ Coverage   84.63%   84.63%   +<.01%     
==========================================
  Files          61       61              
  Lines        4854     4850       -4     
==========================================
- Hits         4108     4105       -3     
  Misses        672      672              
+ Partials       74       73       -1
Impacted Files Coverage Δ
conn/gpio/gpioreg/gpioreg.go 100% <ø> (ø) ⬆️
host/cpu/cpu.go 85.71% <0%> (+3.57%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update 76a7eac...29bbc23. Read the comment docs.

@maruel maruel merged commit bfd820b into master Aug 21, 2017
@maruel maruel deleted the bynumber branch August 21, 2017 12:32
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants