-
-
Notifications
You must be signed in to change notification settings - Fork 71
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
feat: faster overwrites and string min max length support #495
Conversation
@sapphiredev pack this |
Heya @vladfrangu, I've started to run the deployment workflow on this PR at 64f1b90. You can monitor the build here! |
Hey @vladfrangu, I've released this to NPM. You can install it for testing like so: npm install @sapphire/framework@pr-495 |
src/lib/utils/application-commands/ApplicationCommandRegistry.ts
Outdated
Show resolved
Hide resolved
Co-authored-by: Jeroen Claassens <support@favware.tech>
@@ -36,7 +36,9 @@ | |||
"@sapphire/result": "^1.1.1", | |||
"@sapphire/stopwatch": "^1.4.1", | |||
"@sapphire/utilities": "^3.7.0", | |||
"@types/object-hash": "^2.2.1", |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
types in regular deps?
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
yeah otherwise people need to install them themselves or use skipLibCheck and seeing as we're a type-first framework that would be bad DX
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
ah, thanks
I guess BREAKING CHANGE: RegisterBehavior.Overwrite is now the default behaviorWhenNotEqual, and it now uses object hashing instead of checking each property