8000 Add an `impl std::error::Error for mrml::prelude::*::Error` · Issue #36 · jolimail/mrml-core · GitHub
[go: up one dir, main page]
More Web Proxy on the site http://driver.im/
Skip to content
8000
This repository was archived by the owner on Jul 25, 2023. It is now read-only.
This repository was archived by the owner on Jul 25, 2023. It is now read-only.
Add an impl std::error::Error for mrml::prelude::*::Error #36
Closed
@LeoniePhiline

Description

@LeoniePhiline

Hello!

I've been using thiserror and stumbled upon this when trying to integrate mrml:

mrml::prelude::parse::Error and mrml::prelude::render::Error do not implement std::error::Error. That's why they cannot be simply integrated into other error frameworks which all build on the std lib's Error trait.

Here's how this could look like: https://docs.rs/askama_shared/0.12.2/src/askama_shared/error.rs.html#43-54

mrml-core could also opt into using the thiserror crate.

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      0