Skip to content

Conversation

SilkeSchomann
Copy link
Collaborator

Description of work:
To ensure compatibility with both Python 2 and 3, MSlice used the six library. This PR moves it on to Python 3 code instead.

To test:

Code review and observe that tests pass.

Fixes #1039.

Copy link
Contributor

@robertapplin robertapplin left a comment

Choose a reason for hiding this comment

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

Great that we can remove this dependency! Code changes look good

@SilkeSchomann SilkeSchomann merged commit d56a367 into main Dec 10, 2024
2 checks passed
@SilkeSchomann SilkeSchomann deleted the 1039_replace_six branch December 10, 2024 10:38
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
Status: Merged
Development

Successfully merging this pull request may close these issues.

Investigate using Python 3 functionality instead of six
2 participants