Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Appearance settings

enum: remove unused function#132044

Merged
methane merged 1 commit into
python:mainpython/cpython:mainfrom
methane:enum_remove_unusedmethane/cpython:enum_remove_unusedCopy head branch name to clipboard
Apr 8, 2025
Merged

enum: remove unused function#132044
methane merged 1 commit into
python:mainpython/cpython:mainfrom
methane:enum_remove_unusedmethane/cpython:enum_remove_unusedCopy head branch name to clipboard

Conversation

@methane

@methane methane commented Apr 3, 2025

Copy link
Copy Markdown
Member

@python-cla-bot

python-cla-bot Bot commented Apr 6, 2025

Copy link
Copy Markdown

All commit authors signed the Contributor License Agreement.

CLA signed

@methane methane marked this pull request as ready for review April 6, 2025 23:45
@methane methane requested a review from ethanfurman as a code owner April 6, 2025 23:45
@methane methane requested a review from Copilot April 6, 2025 23:45

This comment was marked as off-topic.

@hugovk hugovk left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

History: _dedent was added in #30582 (then briefly reverted in #30632 and re-added in #30637) in v3.11.0a5.

Then its use was removed in #94188 in v3.12.0a1 but the function not removed.

I think it's safe to remove the function: it wasn't documented, was underscore prefixed, is very similar to textwrap.dedent() which should be the first choice, and I don't find any use of _dedent on GitHub code search.

@methane methane merged commit e2b35ee into python:main Apr 8, 2025
@methane methane deleted the enum_remove_unused branch April 8, 2025 00:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants

Morty Proxy This is a proxified and sanitized view of the page, visit original site.