node-rate-limiter-flexible icon indicating copy to clipboard operation
node-rate-limiter-flexible copied to clipboard

Change option name: from inmemoryBlockOnConsumed to inMemoryBlockOnConsumed

Open thernstig opened this issue 5 years ago • 0 comments

Currently the API uses inmemoryBlockOnConsumed as an option, but it would be better if inMemoryBlockOnConsumed.

It is obviously more correct (camelCase), easier to read and it also does not give spelling error with spelling errors tools such https://github.com/streetsidesoftware/cspell

Any chance both inmemoryBlockOnConsumed and inMemoryBlockOnConsumed could be supported, where inMemoryBlockOnConsumed is mentioned in documentation and also takes precedence? I.e. the old name remains usable for backward compability.

thernstig avatar Jan 27 '21 13:01 thernstig