-
Notifications
You must be signed in to change notification settings - Fork 573
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
Suggesting array of categories and string union for LogCategory #6535
Suggesting array of categories and string union for LogCategory #6535
Conversation
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.
It gives good intellisense 👌 And since the addition of duplicate names does not seem unlikely, it could be good to incorporate this change to prevent future breaking changes. It didn't look like the LOG_CATEGORIES
array was publicly exported, which I think is probably important (to not have exported) since users may start relying on the indices which would also break with added categories.
* }); | ||
*/ | ||
static setLogger(loggerCallback: LoggerCallback1): void; | ||
static setLogger(loggerCallback: LoggerCallback2): void; |
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.
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.
b000df9
to
5410e91
Compare
What, How & Why?
This PR is a suggestion to address this comment: #6512 (comment)