Skip to content

Conversation

@numshub
Copy link

@numshub numshub commented Jul 9, 2025

Upgraded code to replace pkg_resources.resource_string with equivalent importlib functionality.

Copy link

@waketzheng waketzheng left a comment

Choose a reason for hiding this comment

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

Should be import importlib.resources as importlib_resources instead of import importlib_resources

from six import binary_type
from six import text_type
import pkg_resources
#import pkg_resources

Choose a reason for hiding this comment

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

Remove it, do not leave it as a comment line. So does line 112 and 113.

Removed old lines of code
Changed import of importlib.resources
@numshub
Copy link
Author

numshub commented Jul 24, 2025

Fixed as requested in review.

@zephvr
Copy link

zephvr commented Jul 25, 2025

This MR seems to be a duplicate of #91

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants