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.

Sign up for GitHub

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

Jump to bottom

Allow fmt=11 in configure #190

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

Allow fmt=11 in configure #190

wants to merge 10 commits into from

Conversation

saraedum
Copy link
Member

@saraedum saraedum commented Aug 7, 2024

No description provided.

@saraedum saraedum enabled auto-merge August 7, 2024 16:01
@saraedum saraedum disabled auto-merge August 7, 2024 16:20
@saraedum saraedum enabled auto-merge August 7, 2024 16:22
@saraedum
Add headers for fmt::join in fmt 11
3df58ef 
not needed for fmt<11 but the ranges.h header was already there in fmt=6
which is the oldest version we support.
@saraedum saraedum closed this Aug 7, 2024
auto-merge was automatically disabled August 7, 2024 17:39

Pull request was closed

@saraedum saraedum reopened this Aug 7, 2024
@saraedum
Copy link
Member Author

saraedum commented Aug 8, 2024

Tests on macOS fail with

   005 EXAMPLES::
  006 
  007     >>> from pyintervalxt import IntervalExchangeTransformation
  008     >>> iet = IntervalExchangeTransformation((18, 3), (1, 0))
  009     >>> iet
  Expected:
      [a: 18] [b: 3] / [b] [a]
  Got:
      [a: ] [b: ] / [b] [a]

@saraedum saraedum mentioned this pull request Aug 8, 2024
Open
@saraedum
Copy link
Member Author

saraedum commented Aug 8, 2024

Bizarre error. I need to actually check this out on a mac.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
1 participant
@saraedum