Skip to content

Store V3 system volumes as Volume enums#290

Merged
bachya merged 1 commit intodevfrom
volumes
Nov 6, 2021
Merged

Store V3 system volumes as Volume enums#290
bachya merged 1 commit intodevfrom
volumes

Conversation

@bachya
Copy link
Copy Markdown
Owner

@bachya bachya commented Nov 6, 2021

Describe what the PR does:

#287 introduce the Volume enum for V3 systems, but didn't go far enough to actually store volume levels as values from this enum. This PR fixes that.

Does this fix a specific issue?

Fixes https://github.com/bachya/python-simplisafe/issues/

Checklist:

  • Confirm that one or more new tests are written for the new functionality.
  • Run tests and ensure everything passes (with 100% test coverage).
  • Update README.md and docs/ with any new documentation.
  • Add yourself to AUTHORS.md.

@bachya bachya added the maintenance Generic library tasks label Nov 6, 2021
@bachya bachya self-assigned this Nov 6, 2021
@qlty-cloud-legacy
Copy link
Copy Markdown

Code Climate has analyzed commit 66da857 and detected 0 issues on this pull request.

View more on Code Climate.

@codecov
Copy link
Copy Markdown

codecov bot commented Nov 6, 2021

Codecov Report

Merging #290 (66da857) into dev (f0e4c0c) will not change coverage.
The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##              dev     #290   +/-   ##
=======================================
  Coverage   99.77%   99.77%           
=======================================
  Files          13       13           
  Lines         904      904           
=======================================
  Hits          902      902           
  Misses          2        2           
Impacted Files Coverage Δ
simplipy/system/v3.py 100.00% <100.00%> (ø)

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 f0e4c0c...66da857. Read the comment docs.

@bachya bachya merged commit 7a098a6 into dev Nov 6, 2021
@bachya bachya deleted the volumes branch November 6, 2021 18:56
bachya added a commit that referenced this pull request Nov 6, 2021
@github-actions github-actions bot locked and limited conversation to collaborators Nov 4, 2022
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

maintenance Generic library tasks

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant