Update compat-table, build data #135

Merged
merged 6 commits into from Jan 11, 2017

Projects

None yet

5 participants

@hzoo
Member
hzoo commented Jan 10, 2017

No description provided.

data/built-ins.json
- },
+ "es7.object.values": {},
+ "es7.object.entries": {},
+ "es7.object.get-own-property-descriptors": {},
@yavorsky
yavorsky Jan 11, 2017 Collaborator

Why we missed these values? 😶

@hzoo
hzoo Jan 11, 2017 Member

Oh I didn't push those for some reason (did it before)

data/built-ins.json
- },
+ "es7.object.values": {},
+ "es7.object.entries": {},
+ "es7.object.get-own-property-descriptors": {},
data/built-ins.json
@@ -35,8 +35,7 @@
"edge": 12,
"firefox": 4,
"safari": 6,
- "node": 0.12,
- "ios": 10
@hzoo
hzoo Jan 11, 2017 Member

need to fix these

hzoo added some commits Jan 10, 2017
@hzoo hzoo update compat-table, ios 537460d
@hzoo hzoo ios6
931edbb
@hzoo hzoo be more conservative ios8/safari8
c368763
@hzoo hzoo fix
59fbbf1
@hzoo hzoo safari8 to ios9
272e31a
@hzoo hzoo cleanup
41f78c5
@hzoo hzoo changed the title from WIP Update compat-table, build data to Update compat-table, build data Jan 11, 2017
@hzoo hzoo requested a review from existentialism Jan 11, 2017
@hzoo
Member
hzoo commented Jan 11, 2017

Updated, only a few went down (ios10 to ios9)

@@ -100,26 +100,31 @@
},
"es6.map": {
"chrome": 51,
+ "firefox": 53,
@chicoxyzzy
chicoxyzzy Jan 11, 2017 edited Collaborator

Not sure that iterator closing is such significant feature for Map and other new types. Other than that (and some other non-significant features of these types) FF can deal with these types since 30/40th versions

@hzoo
hzoo Jan 11, 2017 edited Member

Yeah its not, haven't found a good way to account for these for us cc @ljharb

@hzoo
hzoo Jan 11, 2017 Member

again trying to be correct first/conservative and then we can optimize like in #30

@ljharb
ljharb Jan 11, 2017

Every single feature and observable thing is significant - if something isn't 100% compliant, it's broken, and it needs to be patched/polyfilled/worked around.

@hzoo hzoo merged commit e5aa32d into master Jan 11, 2017

2 checks passed

continuous-integration/travis-ci/pr The Travis CI build passed
Details
continuous-integration/travis-ci/push The Travis CI build passed
Details
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment