{"action":"created","issue":{"url":"https://api.github.com/repos/broccolijs/broccoli/issues/246","labels_url":"https://api.github.com/repos/broccolijs/broccoli/issues/246/labels{/name}","comments_url":"https://api.github.com/repos/broccolijs/broccoli/issues/246/comments","events_url":"https://api.github.com/repos/broccolijs/broccoli/issues/246/events","html_url":"https://github.com/broccolijs/broccoli/issues/246","id":65471960,"number":246,"title":"Broken compatibility for modules that already used the name `rebuild`","user":{"login":"ef4","id":319282,"avatar_url":"https://avatars.githubusercontent.com/u/319282?v=3","gravatar_id":"","url":"https://api.github.com/users/ef4","html_url":"https://github.com/ef4","followers_url":"https://api.github.com/users/ef4/followers","following_url":"https://api.github.com/users/ef4/following{/other_user}","gists_url":"https://api.github.com/users/ef4/gists{/gist_id}","starred_url":"https://api.github.com/users/ef4/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/ef4/subscriptions","organizations_url":"https://api.github.com/users/ef4/orgs","repos_url":"https://api.github.com/users/ef4/repos","events_url":"https://api.github.com/users/ef4/events{/privacy}","received_events_url":"https://api.github.com/users/ef4/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":5,"created_at":"2015-03-31T15:00:50Z","updated_at":"2015-03-31T16:50:41Z","closed_at":null,"body":"It looks like broccoli is trying to detect new-API compatibility based on the presence of a `rebuild` function. But `rebuild` was already a pretty obvious name to use as part of a module's implementation. See for example https://github.com/ef4/ember-browserify/issues/25\r\n\r\nI don't have a solution to propose, I realize the upgrade path is tricky. But I'm sharing this so more people are aware of the pitfall."},"comment":{"url":"https://api.github.com/repos/broccolijs/broccoli/issues/comments/88169109","html_url":"https://github.com/broccolijs/broccoli/issues/246#issuecomment-88169109","issue_url":"https://api.github.com/repos/broccolijs/broccoli/issues/246","id":88169109,"user":{"login":"stefanpenner","id":1377,"avatar_url":"https://avatars.githubusercontent.com/u/1377?v=3","gravatar_id":"","url":"https://api.github.com/users/stefanpenner","html_url":"https://github.com/stefanpenner","followers_url":"https://api.github.com/users/stefanpenner/followers","following_url":"https://api.github.com/users/stefanpenner/following{/other_user}","gists_url":"https://api.github.com/users/stefanpenner/gists{/gist_id}","starred_url":"https://api.github.com/users/stefanpenner/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/stefanpenner/subscriptions","organizations_url":"https://api.github.com/users/stefanpenner/orgs","repos_url":"https://api.github.com/users/stefanpenner/repos","events_url":"https://api.github.com/users/stefanpenner/events{/privacy}","received_events_url":"https://api.github.com/users/stefanpenner/received_events","type":"User","site_admin":false},"created_at":"2015-03-31T16:50:41Z","updated_at":"2015-03-31T16:50:41Z","body":"> npm would provide it with a duplicate different version of Broccoli, so the symbol wouldn't match.\r\n\r\nyes, NPM basically breaks symbols. But it doesn't break branding.\r\n\r\n```\r\nexport default {\r\n  rebuild: Broccoli.rebuild(function() {\r\n\r\n  })\r\n}\r\n```\r\n\r\nwhere `Broccoli.rebuild` decorates the returning function with `isBroccoliRebuild`"}}