Closed Skillz4Killz closed 2 years ago
Latest commit: a9d1e96b93d43599138c144b8a92158e7faf0954
Merging this PR will not cause a version bump for any packages. If these changes should not result in a new version, you're good to go. If these changes should result in a version bump, you need to add a changeset.
Click here to learn what changesets are, and how to add one.
Click here if you're a maintainer who wants to add a changeset to this PR
This is already implemented here https://github.com/guildedjs/guilded.js/blob/main/packages/guilded.js/lib/cache.ts#L12
Please describe the changes this PR makes and why it should be merged:
This PR adds a small bit of customization to the Collection class which will allow us to set a maxSize. This allows supporting cache disabling by simply setting the maxSize to 0, instead of heavy forking and customizing to disabling some parts of cache a bot does not need.
Status
Semantic versioning classification: