Skip to content

Conversation

@alvinometric
Copy link
Contributor

Updated the readme to mention how to grab variants, most of our other SDKs mention it there

@alvinometric alvinometric requested a review from Tymek September 23, 2024 19:27
@coveralls
Copy link

coveralls commented Sep 23, 2024

Coverage Status

coverage: 90.635%. remained the same
when pulling c76956a on alvin/readme-update
into 929f148 on main.

gastonfournier
gastonfournier previously approved these changes Sep 24, 2024
Copy link
Contributor

@gastonfournier gastonfournier left a comment

Choose a reason for hiding this comment

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

LGTM!
cc @melindafekete just for awareness but I think the PR makes sense

melindafekete
melindafekete previously approved these changes Sep 24, 2024
Co-authored-by: Melinda Fekete <[email protected]>
alvinometric and others added 3 commits September 24, 2024 14:29
Co-authored-by: Melinda Fekete <[email protected]>
Co-authored-by: Melinda Fekete <[email protected]>
Co-authored-by: Melinda Fekete <[email protected]>
@alvinometric alvinometric merged commit 2f906a4 into main Sep 24, 2024
@alvinometric alvinometric deleted the alvin/readme-update branch September 24, 2024 17:09
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.

4 participants