-
Notifications
You must be signed in to change notification settings - Fork 443
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
from pkg_resources to importlib_resources #3748
Conversation
@fondbcn please check out your PR's changes. You added a LOT of files to this PR that don't belong here. Please rebase your PR to remove these files. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Please remove the demo site and Pipfile stuff.
nikola/nikola.py
Outdated
@@ -1031,7 +1031,7 @@ def init_plugins(self, commands_only=False, load_all=False): | |||
extra_plugins_dirs = self.config['EXTRA_PLUGINS_DIRS'] | |||
self._loading_commands_only = commands_only | |||
self._plugin_places = [ | |||
resource_filename('nikola', 'plugins'), | |||
importlib_resources('nikola') / 'plugins', |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
- You can’t call a module like that, this code couldn’t have worked.
- We want to use the standard library
importlib.resources
.
Sorry for that, |
Pull Request Checklist
Description
Trying to replace the old pkg_resources with importlib_resources