Skip to content

Conversation

beqqrry-aws
Copy link
Member


By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

@github-actions github-actions bot added the Rust This issue relates to the AWS SDK for Rust (Preview) label Aug 19, 2025

#[derive(Debug, Parser)]
struct Opt {
/// The text to analyze for entities.
Copy link
Contributor

Choose a reason for hiding this comment

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

Meh - these comments sorta suck

.send()
.await?;

println!("Detected entities:");
Copy link
Contributor

Choose a reason for hiding this comment

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

Fine for an example

assert!(result.is_ok(), "Comprehend scenario should complete successfully: {:?}", result);
}

#[tokio::test]
Copy link
Contributor

Choose a reason for hiding this comment

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

These are full integration tests, I'm not sure if I support this. Since they require network access.

Copy link
Member Author

Choose a reason for hiding this comment

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

We have settled on having full integration tests for our examples as a cross-language standard, so this existing does fit the requirements.

}

let primary_language = &languages[0];
let language_code = primary_language.language_code().unwrap_or("en");
Copy link
Contributor

Choose a reason for hiding this comment

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

I don't think this is a correct unwrap

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Rust This issue relates to the AWS SDK for Rust (Preview)
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants