Add case-insensitive table matchers? #17

Closed
shoe opened this Issue Jul 16, 2013 · 3 comments

Projects

None yet

3 participants

@shoe
MojoTech member

No description provided.

@david
MojoTech member

I think an easy solution to this would be to just require that all tables are written in lower case and ensure #to_table always returns lower case. I don't think this is a problem, because I don't think tables should be used to check case correctness. Forcing them to be written all-lower-case would highlight that.

@adsteel

I think always downcasing #to_table might force developers to downcase their tables in the long run, but I think it will be difficult and frustrating in the meantime. Forcing #to_table to lower case essentially makes the table matching case insensitive, but without the developer's awareness. A case-insensitive table matcher seems like a good idea, so that the developer can be more aware of the behavior.

PR opened here #115 and further discussion welcome.

@adsteel

Fulfilled with #121

@adsteel adsteel closed this Sep 28, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment