{"action":"created","issue":{"url":"https://api.github.com/repos/caolan/async/issues/720","labels_url":"https://api.github.com/repos/caolan/async/issues/720/labels{/name}","comments_url":"https://api.github.com/repos/caolan/async/issues/720/comments","events_url":"https://api.github.com/repos/caolan/async/issues/720/events","html_url":"https://github.com/caolan/async/issues/720","id":59025501,"number":720,"title":"suggestion: internally split into separate module files to make optimised builds easier","user":{"login":"jokeyrhyme","id":479816,"avatar_url":"https://avatars.githubusercontent.com/u/479816?v=3","gravatar_id":"","url":"https://api.github.com/users/jokeyrhyme","html_url":"https://github.com/jokeyrhyme","followers_url":"https://api.github.com/users/jokeyrhyme/followers","following_url":"https://api.github.com/users/jokeyrhyme/following{/other_user}","gists_url":"https://api.github.com/users/jokeyrhyme/gists{/gist_id}","starred_url":"https://api.github.com/users/jokeyrhyme/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/jokeyrhyme/subscriptions","organizations_url":"https://api.github.com/users/jokeyrhyme/orgs","repos_url":"https://api.github.com/users/jokeyrhyme/repos","events_url":"https://api.github.com/users/jokeyrhyme/events{/privacy}","received_events_url":"https://api.github.com/users/jokeyrhyme/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":17,"created_at":"2015-02-26T06:15:50Z","updated_at":"2015-05-07T21:23:38Z","closed_at":null,"body":"I just forked this project specifically so I could use `priorityQueue()` and virtually nothing else. It occurs to me that this is a long-term mistake because I'll miss out on future improvements to this function unless I manually import them.\r\n\r\nMay I suggest splitting async.js into multiple files in a lib sub-directory, to facilitate delivering async's functionality to the browser whilst eschewing unnecessary bytes?\r\n\r\nThis may be of little benefit for server-side users, but every byte counts in the front-end.\r\n\r\nAlso, whilst undertaking this pruning, it occurred to me that dependencies between internal and exported methods isn't always clear unless you carefully read the source code. Modularising could assist in making these interdependencies more clear.\r\n\r\nWhat do you think? Should I start a Pull Request for this, or is a bad idea / waste of time?"},"comment":{"url":"https://api.github.com/repos/caolan/async/issues/comments/100023324","html_url":"https://github.com/caolan/async/issues/720#issuecomment-100023324","issue_url":"https://api.github.com/repos/caolan/async/issues/720","id":100023324,"user":{"login":"aearly","id":558190,"avatar_url":"https://avatars.githubusercontent.com/u/558190?v=3","gravatar_id":"","url":"https://api.github.com/users/aearly","html_url":"https://github.com/aearly","followers_url":"https://api.github.com/users/aearly/followers","following_url":"https://api.github.com/users/aearly/following{/other_user}","gists_url":"https://api.github.com/users/aearly/gists{/gist_id}","starred_url":"https://api.github.com/users/aearly/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/aearly/subscriptions","organizations_url":"https://api.github.com/users/aearly/orgs","repos_url":"https://api.github.com/users/aearly/repos","events_url":"https://api.github.com/users/aearly/events{/privacy}","received_events_url":"https://api.github.com/users/aearly/received_events","type":"User","site_admin":false},"created_at":"2015-05-07T21:23:38Z","updated_at":"2015-05-07T21:23:38Z","body":"I don't have commit privileges, I just help out with issue triage. (But it might be worth asking if he's open for more maintainers.)\r\n\r\nThose three steps sound good for a change like this.  But I would still prefer this happen after the backlog of pull-requests is dealt with."}}