Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix #234: Remove attribute sorting #236

Open
wants to merge 4 commits into
base: master
Choose a base branch
from

Conversation

FrankiFixx
Copy link
Contributor

@FrankiFixx FrankiFixx commented Jan 14, 2025

Q A
Is bugfix?
New feature?
Breaks BC?
Fix #234

Copy link

codecov bot commented Jan 14, 2025

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 100.00%. Comparing base (2006c3e) to head (3cce673).

Additional details and impacted files
@@             Coverage Diff             @@
##              master      #236   +/-   ##
===========================================
  Coverage     100.00%   100.00%           
+ Complexity       812       809    -3     
===========================================
  Files             86        86           
  Lines           2221      2215    -6     
===========================================
- Hits            2221      2215    -6     

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

CHANGELOG.md Outdated Show resolved Hide resolved
Co-authored-by: Alexander Makarov <sam@rmcreative.ru>
@samdark samdark requested a review from vjik January 15, 2025 18:03
@vjik vjik changed the title Fix 234 issue Fix #234: Remove attribute sorting Jan 16, 2025
Copy link
Member

@vjik vjik left a comment

Choose a reason for hiding this comment

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

Looks good!

We'll merge it after other BC-break changes are added.

@vjik vjik added the status:ready for merge The pull request is OK to be merged. label Jan 16, 2025
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
status:ready for merge The pull request is OK to be merged.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

Remove attribute sorting
3 participants