Skip to content

Conversation

@blacha
Copy link
Contributor

@blacha blacha commented Nov 8, 2021

Fixes #316

@blacha blacha force-pushed the fix/support-python-enum branch from 9ca8bde to eb36134 Compare November 8, 2021 22:33
@blacha blacha marked this pull request as ready for review November 8, 2021 22:34
@codecov
Copy link

codecov bot commented Nov 8, 2021

Codecov Report

Merging #317 (9126862) into master (527cf79) will increase coverage by 0.01%.
The diff coverage is n/a.

Impacted file tree graph

@@            Coverage Diff             @@
##           master     #317      +/-   ##
==========================================
+ Coverage   81.58%   81.60%   +0.01%     
==========================================
  Files          57       57              
  Lines        5513     5513              
==========================================
+ Hits         4498     4499       +1     
+ Misses       1015     1014       -1     
Impacted Files Coverage Δ
jsonschema/src/error.rs 62.84% <0.00%> (+0.27%) ⬆️

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 527cf79...9126862. Read the comment docs.

Copy link
Owner

@Stranger6667 Stranger6667 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I have a few small readability/naming comments, otherwise, it looks great to me!

Thank you for implementing this :)

Copy link
Owner

@Stranger6667 Stranger6667 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks! Looks good to me! I'll make a new release tomorrow :)

@Stranger6667 Stranger6667 merged commit 0515c95 into Stranger6667:master Nov 9, 2021
@Stranger6667
Copy link
Owner

Released in 0.13.1 🎉

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.

Python enum validation

2 participants