-
-
Notifications
You must be signed in to change notification settings - Fork 2.8k
Allow users to provide version information to the required_plugins INI key #7366
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
Conversation
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.
Lovely, thanks
Friendly ping for a review/merge @nicoddemus and/or @bluetech |
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.
Nice!
@@ -0,0 +1 @@ | |||
Version information as defined by `PEP 440 <https://www.python.org/dev/peps/pep-0440/#version-specifiers>`_ may now be included when providing plugins to the ``required_plugins`` configuration option. |
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.
Possibly we should include this in the changelog which added required_plugins
, since it's the same release? (I'm OK either way)
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.
No strong feelings for either as well, but I think splitting would document the multiple github issues that were used for the required_plugins
feature better ( since we arrange them one issue per line item ). Otherwise, we don't have easy to follow documentation for what closed the linked issue
Bump for a merge @RonnyPfannschmidt @bluetech @nicoddemus |
Closes #7346
Add in the ability to specify versioning information for the
required_plugins
INI key