Skip to content

inspect._signature_strip_non_python_syntax doesn't need to strip / #102378

Closed

Description

@davidhewitt

inspect._signature_strip_non_python_syntax removes /, this is probably a historical thing dating before Python 3.8 made this real Python syntax.

Removing this "feature" would simplify code in inspect.

Linked PRs

Activity

  1. added 2 commits that reference this issue on Mar 2, 2023
  2. terryjreedy commented on Mar 2, 2023

    @terryjreedy
    Member

    Can you provide an example of the difference?

  3. skirpichev commented on Mar 3, 2023

    @skirpichev
    Member

    @terryjreedy, there are adapted tests in the PR.

  4. davidhewitt commented on Mar 3, 2023

    @davidhewitt
    ContributorAuthor

    Yes, the PR hopefully is sufficient demonstration of what I mean by simplification 😊

  5. added a commit that references this issue on Mar 3, 2023
  6. added a commit that references this issue on Mar 9, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions