{"action":"created","issue":{"url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/32","repository_url":"https://api.github.com/repos/sindresorhus/dot-prop","labels_url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/32/labels{/name}","comments_url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/32/comments","events_url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/32/events","html_url":"https://github.com/sindresorhus/dot-prop/issues/32","id":182803732,"number":32,"title":"Consider renaming delete","user":{"login":"LinusU","id":189580,"avatar_url":"https://avatars.githubusercontent.com/u/189580?v=3","gravatar_id":"","url":"https://api.github.com/users/LinusU","html_url":"https://github.com/LinusU","followers_url":"https://api.github.com/users/LinusU/followers","following_url":"https://api.github.com/users/LinusU/following{/other_user}","gists_url":"https://api.github.com/users/LinusU/gists{/gist_id}","starred_url":"https://api.github.com/users/LinusU/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LinusU/subscriptions","organizations_url":"https://api.github.com/users/LinusU/orgs","repos_url":"https://api.github.com/users/LinusU/repos","events_url":"https://api.github.com/users/LinusU/events{/privacy}","received_events_url":"https://api.github.com/users/LinusU/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"assignees":[],"milestone":null,"comments":7,"created_at":"2016-10-13T14:22:27Z","updated_at":"2016-10-14T08:12:15Z","closed_at":null,"body":"`delete` is an operator in javascript and thus is a reserved keyword. That means that you can't have a variable named `delete` in any scope.\r\n\r\nThis also makes this module incompatible with [rollup-plugin-commonjs](https://github.com/rollup/rollup-plugin-commonjs) since you cannot have exports named `delete`.\r\n\r\nI understand if you don't want to make this change, but I think it would be nice to rename the function to e.g. `del`. This would make the module ready for ES6 Modules 🚀 "},"comment":{"url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/comments/253736840","html_url":"https://github.com/sindresorhus/dot-prop/issues/32#issuecomment-253736840","issue_url":"https://api.github.com/repos/sindresorhus/dot-prop/issues/32","id":253736840,"user":{"login":"LinusU","id":189580,"avatar_url":"https://avatars.githubusercontent.com/u/189580?v=3","gravatar_id":"","url":"https://api.github.com/users/LinusU","html_url":"https://github.com/LinusU","followers_url":"https://api.github.com/users/LinusU/followers","following_url":"https://api.github.com/users/LinusU/following{/other_user}","gists_url":"https://api.github.com/users/LinusU/gists{/gist_id}","starred_url":"https://api.github.com/users/LinusU/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/LinusU/subscriptions","organizations_url":"https://api.github.com/users/LinusU/orgs","repos_url":"https://api.github.com/users/LinusU/repos","events_url":"https://api.github.com/users/LinusU/events{/privacy}","received_events_url":"https://api.github.com/users/LinusU/received_events","type":"User","site_admin":false},"created_at":"2016-10-14T08:12:15Z","updated_at":"2016-10-14T08:12:15Z","body":"I have opened an issue here: https://github.com/rollup/rollup-plugin-commonjs/issues/130\r\n\r\nBut I think that this is intended behavior, since setting values on the `exports` object feels to me like you want to export separate things instead of one object.\r\n\r\ni.e.\r\n\r\nCommonJS | ES Modules\r\n----- | -----\r\n`exports.blah = function () {}` | `export function blah () {}`\r\n`module.exports = { ... }` | `export default { ... }`"}}
{"actor":{"display_login":"LinusU"}}