Skip to content

docs: fix inaccuracies in README code examples and descriptions#129

Open
jonathannorris wants to merge 1 commit intomainfrom
fix/readme-accuracy
Open

docs: fix inaccuracies in README code examples and descriptions#129
jonathannorris wants to merge 1 commit intomainfrom
fix/readme-accuracy

Conversation

@jonathannorris
Copy link
Copy Markdown
Member

@jonathannorris jonathannorris commented Apr 15, 2026

Summary

  • Fix code examples in the README that would not compile as written
  • Correct API names, async/await usage, and stale version references
  • Minor grammar and wording fixes

Copy link
Copy Markdown

@gemini-code-assist gemini-code-assist Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Code Review

This pull request updates the README to reflect changes in version 0.3.0 of the OpenFeature Rust SDK, including increasing the minimum Rust version to 1.80.1 and updating API methods like create_client and add_hook. It also corrects asynchronous patterns in examples, such as awaiting the shutdown method and reordering .await calls in flag evaluations. A review comment identifies a missing .await in the get_int_value example, which is necessary for the code to execute correctly.

Comment thread README.md
Signed-off-by: Jonathan Norris <jonathan.norris@dynatrace.com>
Copy link
Copy Markdown

Copilot AI left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Pull request overview

Updates README.md to correct Rust SDK documentation examples so they reflect the current public API and async behavior, aligning stated MSRV and example snippets with the crate’s actual implementation.

Changes:

  • Updated the documented MSRV to 1.80.1 to match Cargo.toml (rust-version).
  • Fixed README code examples to use the correct API/method names and correct async/await ordering.
  • Refreshed stale references in examples (e.g., structured logging version) and corrected a hook trait parameter typo.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread README.md
Comment thread README.md
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.

3 participants