Skip to content

Support max_python_version key in info.json #5594

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

Open
wants to merge 13 commits into
base: V3/develop
Choose a base branch
from

Conversation

plooshi
Copy link

@plooshi plooshi commented Feb 26, 2022

Description of the changes

Make cogs be able to have a maximum python version incase a new version has a change that breaks that cog. (#5590)

@plooshi plooshi requested a review from Jackenmen as a code owner February 26, 2022 16:06
@github-actions github-actions bot added Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Category: Cogs - Downloader This is related to the Downloader cog. Category: Meta This is related to the repository maintenance. labels Feb 26, 2022
@Flame442 Flame442 linked an issue Feb 26, 2022 that may be closed by this pull request
Copy link
Contributor

@Vexed01 Vexed01 left a comment

Choose a reason for hiding this comment

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

I have tested this and can confirm it works as I described in #5590. In the process of testing I noticed the plural/singluar string selection doesn't actually function in the new code and in the min version code - see suggestions.

@plooshi
Copy link
Author

plooshi commented Aug 12, 2022

@Vexed01 i applied the patches you gave in the suggestions.

@Drapersniper
Copy link
Contributor

ensure_max_python_version_info

Can probably be simplified to 3,10 as micro versions shouldn't have any breaking changes.

@Flame442 Flame442 added the Type: Feature New feature or request. label Sep 15, 2022
@Jackenmen Jackenmen force-pushed the V3/feature/max_python_version branch from b4cd663 to 3dca02c Compare December 28, 2022 00:26
@Jackenmen Jackenmen changed the title Maximum python version Support max_python_version key in info.json Dec 28, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Category: Cogs - Downloader This is related to the Downloader cog. Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Category: Meta This is related to the repository maintenance. Type: Feature New feature or request.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[Downloader] max_python_version key in cog info.json
5 participants