No default value for AttibuteValue's `l` field
adamnfish opened this issue · 1 comments
adamnfish commented
A recent change removed the default value for AttributeValue's l field. This means there are heaps of places where I've had to update test code to provide Nil.
I'm struggling to see a reason for this change. If there isn't one, can the default Nil be added? It looks like it may simply be an oversight that this parameter was added without a default. I'm very happy to raise a PR to do so if that seems like a reasonable change?
seratch commented
Default value for l is reasonable for me. I'm looking forward to merging your pull request