Location via proxy:   [ UP ]  
[Report a bug]   [Manage cookies]                
Skip to content
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

Added Edge version that supports exponentiation #1153

Merged
merged 2 commits into from
Feb 26, 2018

Conversation

svijaykoushik
Copy link
Contributor

I've added the version of edge that supports the exponentiation operation which is EDGE 13. I've added this based on this source . It was previously missing in this file.

I've added the version of edge that supports the exponentiation operation which is EDGE 13. I've added this based on this source http://news.thewindowsclub.com/microsoft-introduces-edgehtml-13-80893/
@Elchi3 Elchi3 added the data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript label Feb 21, 2018
@Elchi3
Copy link
Member

Elchi3 commented Feb 21, 2018

I thought this would be Edge 14, c.f. https://developer.microsoft.com/en-us/microsoft-edge/platform/changelog/desktop/14279/

cc'ing @erikadoyle to confirm the correct version here.

@erikadoyle
Copy link
Contributor

Yup, it was introduced in EdgeHTML 14. Thanks @svijaykoushik for adding it in!

@svijaykoushik
Copy link
Contributor Author

@Elchi3 & @erikadoyle according to the change log on Build 10547 of Windows 10 Edge 13 added exponentiation operator as an experimental feature. On Edge 14 they added it in the main stream app. So, my question is which is the right version to add to the repo?

@erikadoyle
Copy link
Contributor

Ah, good point. I think the release version where the feature is enabled by default is probably the most useful. Our features behind experimental flags are subject to code churn / bugs, so probably not reliably stable for mainstream use. (Also they would require end-users to toggle those flags in order to work).

I've updated the correct version number of Edge for exponentiation operator support after review
@svijaykoushik
Copy link
Contributor Author

@erikadoyle Yes, you are correct. The version where the feature is enabled by default would be useful. So, I've changed the version number on the JSON to "14" and committed the change. 🙂

Copy link
Member

@Elchi3 Elchi3 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks @svijaykoushik!

@Elchi3 Elchi3 merged commit 2cacac2 into mdn:master Feb 26, 2018
@erikadoyle
Copy link
Contributor

Thank you @svijaykoushik!

@svijaykoushik svijaykoushik deleted the patch-1 branch March 11, 2018 07:54
@svijaykoushik
Copy link
Contributor Author

Thank you guys for guiding me

@svijaykoushik svijaykoushik restored the patch-1 branch March 11, 2018 07:57
@svijaykoushik svijaykoushik deleted the patch-1 branch March 11, 2018 07:57
dontcallmedom pushed a commit to dontcallmedom/browser-compat-data that referenced this pull request Mar 17, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
data:js 📟 Compat data for JS/ECMAScript features. https://developer.mozilla.org/docs/Web/JavaScript
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants