Description
We added case-insensitive comparison support to various Collections keywords so that keywords got an optional ignore_case
argument (#4343). While doing that, we noticed that four Collections keywords already supported case-insensitivity and had case_insensitive
argument for that purpose. This kind of inconsistency within a library isn't great, so something needs to be done to it.
An easy solution would be using case_insensitive
also with the new keywords. That would make Collections consistent, but because we use ignore_case
in BuiltIn (#2439), different libraries would be inconsistent. I believe we should aim for consistency with all standard libraries and also believe that that ignore_case
is a better argument name than case_insensitive
. It is also consistent with other similar arguments like ignore_order
and ignore_keys
in Collections.
The plan is to do the following:
- Add
ignore_case
argument to Collections keywords that acceptcase_insensitive
. The old argument should be preserved for now, but it should be mentioned in the documentation that the new one is preferred. We shouldn't yet emit deprecation warnings, but we can start doing that in RF 8.0. - For consistency reasons, add
ignore_whitespace
to these same keywords as an alias forwhitespace_insensitive
. It should be treated the same way acase_insensitive
. - For consistency reasons, add
ignore_case
as an alias also for the two String library keywords that acceptcase_insensitive
.
Although we won't be emitting deprecation warnings, it's good to mention this under the deprecated features section in the release notes.