-
-
Notifications
You must be signed in to change notification settings - Fork 4k
chore: Emit deprecation messages #6994
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
Conversation
The output didn't show the names of the methods/getters which may confuse where this was being emitted from.
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.
I'm guessing that the typings already have @deprecated
, LGTM.
Another message can be added as a result of #6889 |
can permission functions warn on usage of the USE_*_THREADS permissions? |
I can for sure check them in |
Please describe the changes this PR makes and why it should be merged:
This pull request emits deprecation warnings to deprecated parts of the library.
This was not done for
MessageEmbed#type
as that was just a property (though it could technically be made into a getter?) and the application events that don't emit at all.Status and versioning classification: