You signed in with another tab or window. Reload to refresh your session.You signed out in another tab or window. Reload to refresh your session.You switched accounts on another tab or window. Reload to refresh your session.Dismiss alert
I was working with gocsv to replace a CSV utility, and I noticed that configuration params were global (e.g. FailIfUnmatchedStructTags). Any reason why we shouldn’t be instancing these values with each Reader (or perhaps a separate Reader)?
Particularly for my use case, we can’t exactly enable these configurations globally since that would break more flexible instances of gocsv decoding that are live with existing external users (in production).
If there isn’t a reason, would love to contribute a new reader that takes those values instanced instead.
The text was updated successfully, but these errors were encountered:
@lumapat I’ve created a new package that does not use global variables, and I have used gocsv as a reference for most of the csv generation process. (Thanks to all gocsv contributors.) I hope the ideas are helpful to you. https://github.com/shigetaichi/xsv
I was working with
gocsv
to replace a CSV utility, and I noticed that configuration params were global (e.g.FailIfUnmatchedStructTags
). Any reason why we shouldn’t be instancing these values with eachReader
(or perhaps a separateReader
)?Particularly for my use case, we can’t exactly enable these configurations globally since that would break more flexible instances of
gocsv
decoding that are live with existing external users (in production).If there isn’t a reason, would love to contribute a new reader that takes those values instanced instead.
The text was updated successfully, but these errors were encountered: