{"action":"created","issue":{"url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/61","repository_url":"https://api.github.com/repos/rbuckton/reflect-metadata","labels_url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/61/labels{/name}","comments_url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/61/comments","events_url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/61/events","html_url":"https://github.com/rbuckton/reflect-metadata/issues/61","id":210380232,"number":61,"title":"Overload signatures must all be ambient or non-ambient.","user":{"login":"masonk","id":449998,"avatar_url":"https://avatars3.githubusercontent.com/u/449998?v=3","gravatar_id":"","url":"https://api.github.com/users/masonk","html_url":"https://github.com/masonk","followers_url":"https://api.github.com/users/masonk/followers","following_url":"https://api.github.com/users/masonk/following{/other_user}","gists_url":"https://api.github.com/users/masonk/gists{/gist_id}","starred_url":"https://api.github.com/users/masonk/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/masonk/subscriptions","organizations_url":"https://api.github.com/users/masonk/orgs","repos_url":"https://api.github.com/users/masonk/repos","events_url":"https://api.github.com/users/masonk/events{/privacy}","received_events_url":"https://api.github.com/users/masonk/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":3,"created_at":"2017-02-27T03:38:33Z","updated_at":"2017-07-11T16:50:59Z","closed_at":null,"body":"On TS 2.1.4, when I import  json-typescript-mapper\": \"^1.1.1\", which depends on reflect-metadata, \r\n\r\nI get many errors similar to this one:\r\n\r\n```(master *%)$ tsc\r\nnode_modules/reflect-metadata/Reflect.ts(157,21): error TS2384: Overload signatures must all be ambient or non-ambient.\r\n```\r\n\r\nThis is the same as issue #37, which is closed. \r\nIt happens for targets es3, es5, and es6.\r\nIt happens for the lowest supported version of reflect-metadata (0.1.3) and the most recent version (0.1.10).\r\n\r\n I am _not_ using Angular. This is basically a brand new project, with json-typescript-mapper and its reflect-metadata as the only deps.\r\n\r\nI think the issue is that you've defined the es7 parts of Reflect as TypeScript, but there are already ambients for the es6 parts of Reflect in the core lib. One solution (which I haven't yet tried, but will soon), is to compile Reflect to .js and emit the types into .d.ts, where I hope they will merge with the core lib types.\r\n\r\nAre you working on this project any more, or did it somehow get folded into core-js? \r\n\r\n```\r\n{\t\r\n\t\"compileOnSave\": true,\r\n    \"compilerOptions\": {\r\n\t\t\"target\": \"es6\",\r\n        \"module\":\"commonjs\",\r\n        \"moduleResolution\": \"node\", //to look up node_modules/@types/**\r\n        // \"target\": \"es6\",\r\n\t\t// \"module\": \"es6\", // default\r\n        \"noImplicitAny\": true,\r\n        \"removeComments\": true,\r\n        \"preserveConstEnums\": true,\r\n\t\t\"sourceMap\": true,\r\n        // \"jsx\": \"react\",\r\n        \"experimentalDecorators\": true,\r\n        \"emitDecoratorMetadata\": true,\r\n        // \"strictNullChecks\": true, // Off for json-typescript-mapper\r\n\r\n        // This must be kept in sync with output.path and resolve.root\r\n        // in dll.webpack.config and dll.webpack.config\r\n\t\t\"outDir\": \"build\",\r\n        // \"typeRoots\" : [\"typings\", \"@types\"],\r\n        \"baseUrl\": \"src\",\r\n        \"paths\": {\r\n            \"*\": [\r\n                \"*\"\r\n            ]\r\n        }\r\n\t},\r\n    \"include\": [\r\n        \"./**/*\"\r\n    ]\r\n    //\"exclude\": [\r\n        // \"exclude\" property defaults to excluding the \r\n        // node_modules, bower_components, and jspm_packages \r\n        // directories when not specified.\r\n    //]\r\n}\r\n```\r\n\r\nPS: Thanks for doing this. This is a neat proposal and I hope it makes it into es7."},"comment":{"url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/comments/314505479","html_url":"https://github.com/rbuckton/reflect-metadata/issues/61#issuecomment-314505479","issue_url":"https://api.github.com/repos/rbuckton/reflect-metadata/issues/61","id":314505479,"user":{"login":"rbuckton","id":3902892,"avatar_url":"https://avatars1.githubusercontent.com/u/3902892?v=3","gravatar_id":"","url":"https://api.github.com/users/rbuckton","html_url":"https://github.com/rbuckton","followers_url":"https://api.github.com/users/rbuckton/followers","following_url":"https://api.github.com/users/rbuckton/following{/other_user}","gists_url":"https://api.github.com/users/rbuckton/gists{/gist_id}","starred_url":"https://api.github.com/users/rbuckton/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/rbuckton/subscriptions","organizations_url":"https://api.github.com/users/rbuckton/orgs","repos_url":"https://api.github.com/users/rbuckton/repos","events_url":"https://api.github.com/users/rbuckton/events{/privacy}","received_events_url":"https://api.github.com/users/rbuckton/received_events","type":"User","site_admin":false},"created_at":"2017-07-11T16:50:59Z","updated_at":"2017-07-11T16:50:59Z","body":"core-js has adopted their own implementation of this. I'm still working on the project, though its been low on my priority list as of late."}}
{"actor":{"display_login":"rbuckton"}}