Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Make _Smallest objects equal again. #5

Merged
merged 2 commits into from
Oct 10, 2018
Merged

Make _Smallest objects equal again. #5

merged 2 commits into from
Oct 10, 2018

Conversation

icemac
Copy link
Member

@icemac icemac commented Oct 10, 2018

This is a regression introduced in 4.1.1 breaking tests in Zope
(e. g. OFS.tests.testOrderSupport.TestOrderSupport.test_orderObjects: sorting by a key leading to an AttributeError reverses the sort order.)

This is a regression introduced in 4.1.1 breaking tests in Zope
(e. g. OFS.tests.testOrderSupport.TestOrderSupport.test_orderObjects)
@icemac icemac self-assigned this Oct 10, 2018
@icemac icemac added this to In progress in Zope 4 final release via automation Oct 10, 2018
Copy link
Member

@jamadden jamadden left a comment

Choose a reason for hiding this comment

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

Assuming CI is green, LGTM in isolation.

CHANGES.rst Outdated
@@ -5,7 +5,8 @@
4.2 (unreleased)
================

- Nothing changed yet.
- Fix regression introduced in 4.1.1 where to `_Smallest` objects are no longer
Copy link
Member

Choose a reason for hiding this comment

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

to -> two

Zope 4 final release automation moved this from In progress to Reviewer approved Oct 10, 2018
[skip ci]
@icemac icemac merged commit 2d9bf6e into master Oct 10, 2018
Zope 4 final release automation moved this from Reviewer approved to Done Oct 10, 2018
@icemac icemac deleted the fix-4.1.1 branch October 10, 2018 15:03
@icemac
Copy link
Member Author

icemac commented Oct 10, 2018

Thank you!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
No open projects
Development

Successfully merging this pull request may close these issues.

None yet

3 participants