`System.global` enumerability

tc39 / proposal-global

ECMAScript Proposal, specs, and reference implementation for `global`

http://tc39.github.io/proposal-global/

MIT License

349 stars 18 forks source link

`System.global` enumerability #14

Closed caridy closed 8 years ago

caridy commented 8 years ago

System is set to grow organically, and even though Object.getOwnPropertyNames(System.global) can be used to inspect System, it will be useful to get consensus about this.

ljharb commented 8 years ago

Arguments for:

System will show global in consoles
Object.assign/Object.{keys,values,entries} works with it
typical reflection methods will show the global

Arguments against:

Object.getOwnPropertyDescriptors/Object.getOwnPropertyNames/for..in works for reflection on System
JSON/Math methods + properties are all non-enumerable - typically builtins are never enumerable.

ljharb commented 8 years ago

Per discussion today, consistency with Math and JSON is paramount - it should stay non-enumerable. (#9)