Skip to content
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.

Sign up for GitHub

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

Jump to bottom

Move private things in modlog and redbot.core.errors #6020

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

Move private things in modlog and redbot.core.errors #6020

wants to merge 2 commits into from

Conversation

Jackenmen
Copy link
Member

@Jackenmen Jackenmen commented Apr 15, 2023 •
edited

Description of the changes

Part of the remainder from #6021 - moving things around in redbot.core.modlog and redbot.core.errors.

Changes are described in the commit titles.

Have the changes in this PR been tested?

Yes

@Jackenmen Jackenmen added Type: Enhancement Something meant to enhance existing Red features. Breaking Change Will potentially break some cogs. labels Apr 15, 2023
@github-actions github-actions bot added Category: Core - API - Config This is related to the `redbot.core.config` module and `redbot.core.drivers` package. Category: Core - Modlog This is related to the core Modlog API. Category: Core - RPC/ZMQ Related to provided (provisional) API for implementing RPC in cogs Category: Core - API - Other This is related to core APIs that don't have a dedicated label. Category: Core - Bot Class This is related to the `redbot.core.bot.Red` class. Category: Core - Bot Commands This is related to core commands (Core and CogManagerUI cog classes). Category: Core - Command-line Interfaces This is related to Red's CLIs (redbot, redbot-launcher, redbot-setup). Category: Core - Other Internals This is related to core internals that don't have a dedicated label. Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Category: Meta This is related to the repository maintenance. labels Apr 15, 2023
@Jackenmen Jackenmen force-pushed the privatize_easy_things branch 2 times, most recently from 8ad7a1e to f010ffb Compare April 15, 2023 21:09
@Jackenmen Jackenmen added this to the 3.5.1 milestone Apr 15, 2023
@Jackenmen Jackenmen mentioned this pull request May 3, 2023
Merged
@Jackenmen Jackenmen removed the Breaking Change Will potentially break some cogs. label May 8, 2023
@github-actions github-actions bot removed Category: Core - Bot Class This is related to the `redbot.core.bot.Red` class. Category: Core - RPC/ZMQ Related to provided (provisional) API for implementing RPC in cogs Category: Docs - Other This is related to documentation that doesn't have its dedicated label. Category: Core - Other Internals This is related to core internals that don't have a dedicated label. Category: Core - Command-line Interfaces This is related to Red's CLIs (redbot, redbot-launcher, redbot-setup). Category: Meta This is related to the repository maintenance. labels May 8, 2023
@Jackenmen Jackenmen changed the title Privatize things that shouldn't **and aren't** be used outside Red Prefix modlog.handle_auditype_key with _ and move MissingExtraRequirements out of public namespace May 8, 2023
@Jackenmen Jackenmen changed the title Prefix modlog.handle_auditype_key with _ and move MissingExtraRequirements out of public namespace Move private things in modlog and redbot.core.errors May 8, 2023
@Jackenmen Jackenmen modified the milestones: 3.5.3, 3.5.2 May 11, 2023
@Jackenmen Jackenmen modified the milestones: 3.5.2, 3.6.0, 3.5.3 May 12, 2023
@Jackenmen Jackenmen modified the milestones: 3.5.6, 3.5.7 Mar 18, 2024
@Jackenmen Jackenmen modified the milestones: 3.5.9, 3.5.10 Apr 20, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
Category: Core - API - Config This is related to the `redbot.core.config` module and `redbot.core.drivers` package. Category: Core - API - Other This is related to core APIs that don't have a dedicated label. Category: Core - Bot Commands This is related to core commands (Core and CogManagerUI cog classes). Category: Core - Modlog This is related to the core Modlog API. Type: Enhancement Something meant to enhance existing Red features.
Projects
None yet
Milestone
3.5.10
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@Jackenmen