search

XAMPPRocky / fluent-templates

Easily add Fluent to your Rust project.
Apache License 2.0
136 stars 28 forks source link

feat: Remove panic, prefer error handling #63

Closed Syndelis closed 3 months ago

Syndelis commented 5 months ago

Resolves #54

This PR swaps the panic call for proper error handling. The function now returns a Result with a specialized error type, rather than an Option.

Tests for these new errors are still TODO; I just wanted to collect some feedback on this first.

XAMPPRocky commented 3 months ago

Thank you for your PR, and congrats on your first contribution! 🎉