Skip to content

Tidy check for io::ErrorKind::Other in std code #102122

Open
@Mark-Simulacrum

Description

@Mark-Simulacrum

We have a note in the ErrorKind documentation:

This ErrorKind is not used by the standard library.

but in practice, there are cases were we do make use of it, whether in tests or by accident. We should add a check in tidy that the ErrorKind::Other string doesn't show up in std code, with an ability to opt-out similar to style checks (probably on the same line only would be best).

The ideal of course is a compiler check but I don't think there's much we can do there without a custom lint, which seems like way too much effort. The tidy check can miss things (e.g., glob imports), but that's OK.

Metadata

Metadata

Assignees

No one assigned

    Labels

    A-testsuiteArea: The testsuite used to check the correctness of rustcT-infraRelevant to the infrastructure team, which will review and decide on the PR/issue.

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions