Skip to content

show_error_context does not work in config file #2964

Closed

Description

@tyehle

Looks like the name was changed from hide_error_context, but was unchanged internally. Since the config file settings are copied directly into options the following ini will fail with mypy.ini: [mypy]: Unrecognized option: show_error_context = True

[mypy]
show_error_context = True

hide_error_context = False works as expected.

Possible fix: refactor the internal name to match the external one.

Activity

  1. gvanrossum commented on Mar 5, 2017

    @gvanrossum
    Member

    Thanks, good catch. Do you feel comfortable sending a PR yourself?

  2. tyehle commented on Mar 5, 2017

    @tyehle
    ContributorAuthor

    Probably, but from #2545 it looks like the name wasn't changed for backward compatibility.

  3. gvanrossum commented on Mar 5, 2017

    @gvanrossum
    Member

    Hm, this comment suggests that we thought about changing it but then dropped the thought. The docs state that it's named show_error_context, and I think that should prevail, regardless of backwards compatibility.

  4. added a commit that references this issue on Mar 6, 2017
    7f97f6e
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

    bugmypy got something wrong

    Projects

    No projects

      Milestone

      No milestone

      Relationships

      None yet

      Development

      No branches or pull requests

      Issue actions