Skip to content

Make the recently added compat module standalone. #74

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

Closed
wants to merge 1 commit into from

Conversation

albu-diku
Copy link
Contributor

Work since the introduction of ensure_native_string() has shown that being able to make use the function is limited by its inclusion of the base package. Allow the function to be used everywhere by using only the Python standard library for its operation.

This sidesteps any potential for circular imports and ensures well tested string conversion is available for the definition of e.g. base itself.

Work since the introduction of ensure_native_string() has shown that being
able to make use the function is limited by its inclusion of the base
package. Allow the function to be used everywhere by using only the Python
standard library for its operation.

This sidesteps any potential for circular imports and ensures well tested
string conversion is available for the _definition_ of e.g. base itself.
@albu-diku albu-diku force-pushed the fix/no-dependency-compat branch from e252e6d to 829975d Compare July 3, 2024 10:05
@jonasbardino jonasbardino added enhancement New feature or request unit test labels Jul 3, 2024
@jonasbardino
Copy link
Contributor

Thanks @albu-diku. Merged through svn with minor adjustments.
Mainly added a few docstrings/comment and copied our explanation for going against PEP8 in the use of type comparisons instead of isinstance in unicode string detection. It originates from the similar helper in mig.shared.base and basically boils down to compatibility issues across python versions with future and futurize in the mix.

@albu-diku albu-diku deleted the fix/no-dependency-compat branch July 19, 2024 08:36
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants