This repository has been archived by the owner on May 27, 2024. It is now read-only.
update rust binary module to current standards #289
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
I saw that the rust example module was in a really bad shape, so I updated it to better fit current standards.
Changes include:
extern crate
, macros are imported usinguse
statements)std::fs::read_to_string
anyhow
as a dependency to help with error handling (there are other ways to do that, but those would either require a lot more boilerplate or remove context from the errors)All in all, main.rs now has half the amount of lines and is much closer to idiomatic Rust.
If there are any other changes I should make to this, please tell me. I'm also thinking about writing an ansible_module library for Rust, streamlining the experience further, but I think this is a good first step.