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.

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

Define a type to represent any Ion Value #46

Open
R-maan opened this issue May 26, 2020 · 0 comments
Open

Define a type to represent any Ion Value #46

R-maan opened this issue May 26, 2020 · 0 comments
Labels
enhancement New feature or request M Medium T-Shirt size

Comments

@R-maan
Copy link
Contributor

R-maan commented May 26, 2020

I do not understand why we need a more formal type for this. ionItem is being used only for this test, should we create a public type?

Sorry, to clarify: I don't think we need a more formal type for this--it's fine as-is. However, I expect that at some point in the (distant?) future, ion-go will have a public-facing formal type that represents any Ion value. (Similar to ion-java's IonValue class.) If/when we add that, we should replace this so we're not maintaining two data types for the same thing.

Originally posted by @zslayton in #33

@therapon therapon added enhancement New feature or request M Medium T-Shirt size labels Jun 8, 2020
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request M Medium T-Shirt size
Projects
None yet
Development

No branches or pull requests

2 participants