-
-
Notifications
You must be signed in to change notification settings - Fork 3.1k
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
Backport #12667 on branch 3.6.x (Add events service) #13465
Backport #12667 on branch 3.6.x (Add events service) #13465
Conversation
Thanks for making a pull request to jupyterlab! |
2ca872c
to
b0d81ca
Compare
@@ -16,7 +16,7 @@ | |||
"resolveJsonModule": true, | |||
"sourceMap": true, | |||
"strictNullChecks": true, | |||
"target": "es2017", | |||
"target": "es2018", |
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.
@afshin are we ok to change that for a minor release?
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 am not sure I fully understand the ramifications. I don’t think this is backward-incompatible but I’m not 100% confident.
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.
This is backward incompatible in the sense that it raises the required version of web browsers.
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 think in that sense we might be okay because all the browsers we support are covered.
Do you think we have a problem?
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 guess the time of holding back on web browser update in organization is mostly over. But it is worth raising the point at a weekly call (although this week may not be the best).
Backport #12667