bottlerocket-os / bottlerocket-os/bottlerocket-settings-sdk

Move away from `snafu` for `pub` interfaces

Open
#32 1 comment 1 reaction 0 assignees View on GitHub
enhancement rust
Dominant language
Rust
Stars
5
Forks
36
Avg merge
4d 12h
Merged PRs (30d)
2

Description

Per a comment from @webern on #31:

> We are exposing a library and its traits in our public interface and it might be impossible to avoid a major version break if the Snafu library changes in some way.

We may wish to more closely control the fate of our public API by using our own error types, either across the board or by converting to them at the last moment for any `pub` methods. The downside to the "JIT"-style conversions is that it's not easy to create a mechanism to check this for us at compile-time.

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.