plugins: cleanup and rename type to group#5412
plugins: cleanup and rename type to group#5412radoering merged 1 commit intopython-poetry:masterfrom
Conversation
radoering
left a comment
There was a problem hiding this comment.
test_plugin_manager.py still uses "plugin" and type instead of group. Maybe, this should be changed, too.
|
I'm not sure whether this PR is to blame, but if I do $ poetry plugin show
KeyError
'export'
at ~/.local/share/pypoetry/venv/lib/python3.8/site-packages/poetry/console/commands/plugin/show.py:55 in handle
51│ category = "plugins"
52│ if issubclass(plugin, ApplicationPlugin):
53│ category = "application_plugins"
54│
→ 55│ package = packages_by_name[canonicalize_name(entry_point.name)]
56│ plugins[package.pretty_name]["package"] = package
57│ plugins[package.pretty_name][category].append(entry_point)
58│
59│ for name, info in plugins.items():This is after installing the export plugin. |
|
@lovesegfault it could be. But I'm not sure. I'd recommend that you consider waiting till #5413 is merged. Also note that the export plugin is now also renamed. The old one will only work with poetry I do expect another PR after that to look at the commands. If it remains an issue with the above PR, please create a new issue. |
|
@lovesegfault had a quick look, and no that issue is not because of this PR. The root cause here is that poetry currently expects the entrypoint name to be the same as the package name. Not entirely sure if this was intentional. Since the export plugin is not named If i replace $ poetry plugin show
• poetry-plugin-export (1.0.0) Poetry plugin to export the dependencies to various formats
1 application plugin
Dependencies
- poetry (>=1.2.0b1dev0,<2.0.0) |
|
This pull request has been automatically locked since there has not been any recent activity after it was closed. Please open a new issue for related bugs. |
Minor improvements to plugin implementation to make downstream implementations a bit easier for application plugins. The main change here is the renaming of
typeproperty togroup. This reflects reality more as this is referring to the entrypoint group the plugin is made available under and avoids shadowing built-intype.