Beat comments

Results 33 comments of


                                            Beat

configuration_method bug

Thanks for raising this. I'm not sure what the usecase is to configure a method on an otherwise unused scope, can you provide an example?

configuration_method bug

Sounds interesting - can you provide an example usage in code?

Instance-level configuration

Interesting - could you elaborate on the use case a little bit more, specifically how this would be different from instantiating classes in your gem with parameters? e.g. : ```...

Option to return atoms instead of strings for keys

Hi @drernie apologies for the late reply - could you outline what the use case for this is? I agree with @hauleth that we have to be careful with casting...

Feature/add raw line on error tuple option

Thanks for putting this together I'll hopefully get a look later this week

Feature/add raw line on error tuple option

Hey @y86 this is looking good, thanks! I think adding more information on errors is going to help users of this library get to the bottom of problems more quickly....

Option to include original csv line in resulting tuple returned by CSV.decode(stream, options)

Hi, this is interesting. Could definitely start something on this, but would also accept a PR to make this optional behaviour.

Option to include original csv line in resulting tuple returned by CSV.decode(stream, options)

Maybe have a look at pr https://github.com/beatrichartz/csv/pull/95 - this might be what you are looking for

CSV.decode! add additional \r\n to double quoted value containing \n

Hi, this is current behaviour since this library operates line by line. Will fix this in upcoming versions.

Adding an option to set up the quote_char

Interesting, thanks for sharing - Your usecase is a "stray quote", which is considered illegal [as per RFC4180](https://tools.ietf.org/html/rfc4180#page-3) - I have seen them in quite a few files in the...