Skip to content

Update the API to be more resilient#39

Open
rcloran wants to merge 6 commits intobparli:masterfrom
rcloran:api
Open

Update the API to be more resilient#39
rcloran wants to merge 6 commits intobparli:masterfrom
rcloran:api

Conversation

@rcloran
Copy link
Copy Markdown
Contributor

@rcloran rcloran commented Sep 13, 2023

  • Accept types that don't need parsing/interpreting where possible
    • No strings for IpAddr
    • Take Duration instead of u64 to clarify meaning
  • Add an Error enum, and return variants of it instead of String
  • Add documentation

Fixes #30

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

(Also?) expose IpAddr in the API directly

1 participant