Skip to content

Conversation

@ahopkins
Copy link

Putting all of the transition strings into an enum should make it easier to manage going forward.

@codecov
Copy link

codecov bot commented Mar 14, 2021

Codecov Report

Merging #35 (779b95a) into actor (fe70a97) will increase coverage by 0.25%.
The diff coverage is 100.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##            actor      #35      +/-   ##
==========================================
+ Coverage   94.54%   94.79%   +0.25%     
==========================================
  Files           5        5              
  Lines         458      480      +22     
  Branches       62       62              
==========================================
+ Hits          433      455      +22     
  Misses         18       18              
  Partials        7        7              
Flag Coverage Δ
unittests 94.37% <100.00%> (+0.27%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
jumpstarter/states.py 97.93% <100.00%> (+0.26%) ⬆️

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 fe70a97...779b95a. Read the comment docs.

@ahopkins ahopkins changed the title Move tranitions to enum Move transitions to enum Mar 14, 2021
@ghost
Copy link

ghost commented Mar 14, 2021

Congratulations 🎉. DeepCode analyzed your code in 3.362 seconds and we found no issues. Enjoy a moment of no bugs ☀️.

👉 View analysis in DeepCode’s Dashboard | Configure the bot

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.

3 participants