Bug #21216
closed
Fix "memoryRetryMultipler" misspelled name
Added by Brett Smith 12 months ago.
Updated 8 months ago.
Release relationship:
Auto
Description
This is in our CWL OutOfMemoryRetry extension. "Multipler" is not a word. I don't know whether we meant "Multiple" or "Multiplier".
I don't know what the backwards compatibility story is here, that should be specified before development. But when you're writing code in YAML with little tooling support to catch errors, predictable names go a long way, so I think this is worth fixing somehow.
That's embarrassing. Yes, that was supposed to be "Multiplier".
We can fix it to accept both (the misspelled value and the new one) and have the documentation tell people to use the correct spelling.
- Target version changed from Future to Development 2024-01-17 sprint
- Assigned To set to Peter Amstutz
21216-multiplier-typo-fix @ 999ef5f71d522283465b54b6468bae6badf28ca4
developer-run-tests: #3991
- All agreed upon points are implemented / addressed.
- Anything not implemented (discovered or discussed during work) has a follow-up story.
- Code is tested and passing, both automated and manual, what manual testing was done is described
- yes, updated the tests with the new, fixed spelling and added a regression test with the old spelling
- Documentation has been updated.
- yes, only lists the new spelling
- Behaves appropriately at the intended scale (describe intended scale).
- Considered backwards and forwards compatibility issues between client and server.
- accepts both spellings, with the correct spelling taking precedence
- Follows our coding standards and GUI style guidelines.
- Status changed from New to In Progress
- Target version changed from Development 2024-01-17 sprint to Development 2024-01-31 sprint
- Target version changed from Development 2024-01-31 sprint to Development 2024-02-14 sprint
- Status changed from In Progress to Resolved
- Tracker changed from Idea to Bug
Also available in: Atom
PDF