-
Notifications
You must be signed in to change notification settings - Fork 2
This extension must be completely revised. #4
Comments
You're right on the part about symbols not being able to start with a number, that was an oversight on my part. However I'd say it's a bit extreme to demand that everything should be revised based off of that. As for your second point, if doing this would cause issues with your code, (Doesn't in mine) then just don't run the command on the affected lines. Simple. |
It's surprising that this extension subtly alters the behavior of hashes that have string keys. This could cause people who think that the altered code is semantically equivalent to introduce bugs into their code. I think a more conservative approach—applying the transformation only to keys that are symbols—would lead to less wailing and gnashing of teeth. That's how this Emacs package behaves. |
@patbl Would this be a more suitable change for you? #11 The result would now look like this: data = [
{ 1: "en", 2: "to", 3: "tre" },
{ one: "un", two: "deux", three: "trois" },
{ "weights" => ["pound", "kilogram"], "lengths" => ["meter", "mile"] },
{ de: { one: "eins", two: "zwei", three: "drei" } }
] |
When you say run the command, what command is that? |
The example in README.md denote a poor knowledge of ruby language.
The text was updated successfully, but these errors were encountered: