-
Notifications
You must be signed in to change notification settings - Fork 237
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
- AppInsightsCore: Enable setting NotificationManager during initialization #1331
Conversation
MSNev
commented
Jul 29, 2020
- Refactor code to provide better tree shaking and minification of generated code Refactor code to provide better tree shaking and minification of generated code #1076
0319652
to
39375a5
Compare
…zation - Refactor code to provide better tree shaking and minification of generated code #1076
39375a5
to
25b26ed
Compare
@@ -158,7 +158,7 @@ export class CoreUtils { | |||
* @param {any} value - Value to be checked. | |||
* @return {boolean} True if the value is a boolean, false otherwise. | |||
*/ | |||
public static isBoolean(value: any): boolean { | |||
public static isBoolean(value: any): value is boolean { |
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.
cool!
const subject = this.getSubject(initializer, name); | ||
// this.name = name; | ||
// this.initializer = initializer; | ||
// } |
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.
will all of these be added back in?
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.
As some point we should... But as they where never used I just commented them out for now over just straight out deleting.