Skip to content

Conversation

@fgmacedo
Copy link
Owner

@fgmacedo fgmacedo commented Nov 16, 2024

  • feat: Adding State.from_.any()

Relates to: #494

@fgmacedo fgmacedo changed the title macedo/from any Adding State.from_.any() Nov 16, 2024
@codecov
Copy link

codecov bot commented Nov 17, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (bf55631) to head (5828cbd).
Report is 15 commits behind head on develop.

Additional details and impacted files
@@            Coverage Diff            @@
##           develop      #497   +/-   ##
=========================================
  Coverage   100.00%   100.00%           
=========================================
  Files           23        23           
  Lines         1566      1593   +27     
  Branches       249       253    +4     
=========================================
+ Hits          1566      1593   +27     
Flag Coverage Δ
unittests 100.00% <100.00%> (ø)

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

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


def add_from_attributes(cls, attrs): # noqa: C901
for key, value in sorted(attrs.items(), key=lambda pair: pair[0]):
for key, value in attrs.items():
Copy link
Owner Author

Choose a reason for hiding this comment

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

The library is pre-python3.7 where the dictionary order was not guaranteed. This order hack is not required since 3.7.

@sonarqubecloud
Copy link

@fgmacedo fgmacedo added this to the 2.4.1 milestone Nov 18, 2024
@fgmacedo fgmacedo merged commit c1fea89 into develop Nov 18, 2024
12 checks passed
@fgmacedo fgmacedo deleted the macedo/from_any branch November 18, 2024 21:07
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.

2 participants