Skip to content

Update no-panics lint to perform checks by itself, removing the dependency on clippy #9

@KaiserKarel

Description

@KaiserKarel

Currently the no-panics lint checks that clippy has been configured in a way to deny all panics. That creates a bit of boilerplatey noise, as we add the same attributes at the top of each crate. We instead want to import the no-panics related lints from clippy and register them in substrace, so that we can avoid checking for existing attributes and instead perform the checks ourselves.

Currently the lints that we want to import are:

  • disallowed_methods
  • indexing_slicing
  • todo
  • unwrap_used
  • panic

If possible, we'd like to still allow these in code when compiling under cfg(test), but if that cannot be easily implemented, not needed.

Metadata

Metadata

Assignees

No one assigned

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions