Skip to content

Handle cog requirement reinstall in redbot-update#6756

Merged
Jackenmen merged 3 commits into
Cog-Creators:V3/developfrom
Jackenmen:automatically_reinstall_cog_reqs
May 25, 2026
Merged

Handle cog requirement reinstall in redbot-update#6756
Jackenmen merged 3 commits into
Cog-Creators:V3/developfrom
Jackenmen:automatically_reinstall_cog_reqs

Conversation

@Jackenmen

@Jackenmen Jackenmen commented May 16, 2026

Copy link
Copy Markdown
Member

Description of the changes

I forgot to do this when this command was introduced in #6734.

Have the changes in this PR been tested?

Yes

@Jackenmen Jackenmen added this to the 3.5.25 milestone May 16, 2026
@Jackenmen Jackenmen added Type: Enhancement Something meant to enhance existing Red features. Changelog Entry: Skipped Changelog entry for this PR is unnecessary. labels May 16, 2026
@github-actions github-actions Bot added the Category: Core - Command-line Interfaces This is related to Red's CLIs (redbot, redbot-launcher, redbot-setup). label May 16, 2026
@Jackenmen

Jackenmen commented May 18, 2026

Copy link
Copy Markdown
Member Author

This has now been tested for the following cases:

  • Python version changed during update (I also tested that Red doesn't remove the lib on startup, if it already got regenerated)
    image
  • Python version did not change during update
    image

And everything works as expected.

@Flame442 Flame442 self-assigned this May 25, 2026
@Jackenmen Jackenmen merged commit 77457e9 into Cog-Creators:V3/develop May 25, 2026
18 checks passed
@Jackenmen Jackenmen deleted the automatically_reinstall_cog_reqs branch May 25, 2026 20:11
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Category: Core - Command-line Interfaces This is related to Red's CLIs (redbot, redbot-launcher, redbot-setup). Changelog Entry: Skipped Changelog entry for this PR is unnecessary. Type: Enhancement Something meant to enhance existing Red features.

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants