Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Refactor errors (breaking) #33

Merged
merged 10 commits into from
Sep 28, 2021
Merged

Refactor errors (breaking) #33

merged 10 commits into from
Sep 28, 2021

Commits on Sep 27, 2021

  1. refactor: ♻️ switched to thiserror and anyhow in the CLI

    Error reporting is now much nicer.
    arctic-hen7 committed Sep 27, 2021
    Configuration menu
    Copy the full SHA
    59599fe View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    a1bfc6e View commit details
    Browse the repository at this point in the history
  3. Configuration menu
    Copy the full SHA
    2daabb1 View commit details
    Browse the repository at this point in the history

Commits on Sep 28, 2021

  1. Configuration menu
    Copy the full SHA
    5340d16 View commit details
    Browse the repository at this point in the history
  2. Configuration menu
    Copy the full SHA
    e8ca3f4 View commit details
    Browse the repository at this point in the history
  3. fix: 🐛 fixed error #[from] for ClientError

    Was `FetchError`, should be `ClientError`.
    arctic-hen7 committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    ca04f12 View commit details
    Browse the repository at this point in the history
  4. feat: ✨ made error formatting use fmterr

    We don't need `anyhow` anymore.
    arctic-hen7 committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    033fca8 View commit details
    Browse the repository at this point in the history
  5. feat(templates): ✨ allowed usage of any error type in render functions

    Before, only `String` was allowed as an error type.
    This also makes `?` usable.
    
    BREAKING CHANGE: `StringResult`/`StringResultWithCause` are replaced by `RenderFnResult`/`RenderFnResultWithCause`
    arctic-hen7 committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    19e685b View commit details
    Browse the repository at this point in the history
  6. fix: 🐛 fixed newlines/tabs in initial state causing serialization errors

    We're now using JS raw strings, escaping as necessary, and then escaping control characters in the shell.
    arctic-hen7 committed Sep 28, 2021
    Configuration menu
    Copy the full SHA
    4b933c7 View commit details
    Browse the repository at this point in the history
  7. Configuration menu
    Copy the full SHA
    b84c14c View commit details
    Browse the repository at this point in the history