search

nestjs / docs.nestjs.com

The official documentation https://docs.nestjs.com 📕
MIT License
1.18k stars 1.69k forks source link

Please document that the default is 100 items max for cache-manager #3074

Open falatime opened 1 month ago

falatime commented 1 month ago

Is there an existing issue that is already proposing this?

Is your feature request related to a problem? Please describe it

No documentation that there is a limit of 100 items https://github.com/nestjs/cache-manager/blob/master/lib/default-options.ts#L3

Describe the solution you'd like

No documentation that there is a limit of 100 items https://github.com/nestjs/cache-manager/blob/master/lib/default-options.ts#L3

Teachability, documentation, adoption, migration strategy

No response

What is the motivation / use case for changing the behavior?

No documentation that there is a limit of 100 items https://github.com/nestjs/cache-manager/blob/master/lib/default-options.ts#L3

micalevisk commented 1 month ago

PRs are more than welcomed