Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

It would be slightly more idiomatic and user-friendly to return the Type from Next() and have one of the possible values indicate error. #28

Open
therapon opened this issue May 17, 2020 · 2 comments
Open

It would be slightly more idiomatic and user-friendly to return the Type from Next() and have one of the possible values indicate error. #28

therapon opened this issue May 17, 2020 · 2 comments
Assignees
@R-maan
Labels
API Related to public API for the library L Large T-Shirt size

Comments

@therapon
Copy link
Contributor

It would be slightly more idiomatic and user-friendly to return the Type from Next() and have one of the possible values indicate error.

Originally posted by @mrutter-amzn in #26
@therapon therapon mentioned this issue May 17, 2020
Merged
@R-maan
Copy link
Contributor

R-maan commented May 21, 2020

I am looking into it.

@R-maan
Copy link
Contributor

R-maan commented May 21, 2020

More details and discussion: #26 (comment)

@therapon therapon added API Related to public API for the library L Large T-Shirt size and removed API Related to public API for the library labels Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@R-maan R-maan

Labels
API Related to public API for the library L Large T-Shirt size
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@therapon @R-maan