Skip to content

Use enums instead of literals - #81

Merged
robin-nitrokey merged 1 commit into
mainfrom
enums
Nov 17, 2023
Merged

Use enums instead of literals#81
robin-nitrokey merged 1 commit into
mainfrom
enums

Conversation

@robin-nitrokey

Copy link
Copy Markdown
Member

Using enums makes it easier to use the functions in a typesafe way and to validate user input.

Fixes: #69

@codecov

codecov Bot commented Nov 17, 2023

Copy link
Copy Markdown

Codecov Report

Attention: 22 lines in your changes are missing coverage. Please review.

Comparison is base (bcaaedb) 79.54% compared to head (f28c8c5) 79.43%.
Report is 1 commits behind head on main.

❗ Current head f28c8c5 differs from pull request most recent head 23eb940. Consider uploading reports for the commit 23eb940 to get more accurate results

Files Patch % Lines
nethsm/__init__.py 56.00% 22 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #81      +/-   ##
==========================================
- Coverage   79.54%   79.43%   -0.12%     
==========================================
  Files         751      751              
  Lines       13255    13292      +37     
==========================================
+ Hits        10544    10558      +14     
- Misses       2711     2734      +23     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

Using enums makes it easier to use the functions in a typesafe way and
to validate user input.

Fixes: #69
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Use enums instead of string literals

1 participant