GitHub PullRequestReviewCommentEvent at 2014-04-01 10:14:15
Repository: cnpmjs.org
Actor: fengmk2
Payload: {"comment":{"url":"https://api.github.com/repos/cnpm/cnpmjs.org/pulls/comments/11154371","id":11154371,"diff_hunk":"@@ -108,10 +109,7 @@ var config = {\n // load config/config.js, everything in config.js will cover the same key in index.js\n var customConfig = path.join(root, 'config/config.js');\n if (fs.existsSync(customConfig)) {\n-  var options = require(customConfig);\n-  for (var k in options) {\n-    config[k] = options[k];\n-  }\n+  copy(require(customConfig)).toCover(config);","path":"config/index.js","position":16,"original_position":16,"commit_id":"b041cc736137d863152aa1a524998e6761e9b75c","original_commit_id":"b041cc736137d863152aa1a524998e6761e9b75c","user":{"login":"fengmk2","id":156269,"avatar_url":"https://avatars.githubusercontent.com/u/156269?","gravatar_id":"95b9d41231617a05ced5604d242c9670","url":"https://api.github.com/users/fengmk2","html_url":"https://github.com/fengmk2","followers_url":"https://api.github.com/users/fengmk2/followers","following_url":"https://api.github.com/users/fengmk2/following{/other_user}","gists_url":"https://api.github.com/users/fengmk2/gists{/gist_id}","starred_url":"https://api.github.com/users/fengmk2/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/fengmk2/subscriptions","organizations_url":"https://api.github.com/users/fengmk2/orgs","repos_url":"https://api.github.com/users/fengmk2/repos","events_url":"https://api.github.com/users/fengmk2/events{/privacy}","received_events_url":"https://api.github.com/users/fengmk2/received_events","type":"User","site_admin":false},"body":"https://github.com/node-modules  \r\n\r\n还有很多  \r\n\r\n----\r\n\r\n@fengmk2 (https://twitter.com/fengmk2)\r\n@Python发烧友 (http://weibo.com/imk2/profile)\r\nhttp://fengmk2.github.com (http://fengmk2.github.com/)\r\n\r\n---------------------------\r\n*记得当时年纪小/\r\n你爱谈天，我爱笑/\r\n有一回并肩坐在桃树下/\r\n风在林梢鸟儿在叫/\r\n我们不知怎样睡着了/\r\n梦里花落知多少*\r\n\r\n\r\nOn Tuesday, April 1, 2014 at 6:13 PM, dead_horse wrote:\r\n\r\n> In config/index.js:\r\n> > @@ -108,10 +109,7 @@ var config = { > // load config/config.js, everything in config.js will cover the same key in index.js > var customConfig = path.join(root, 'config/config.js'); > if (fs.existsSync(customConfig)) { > - var options = require(customConfig); > - for (var k in options) { > - config[k] = options[k]; > - } > + copy(require(customConfig)).toCover(config);  \r\n> wowjs吧.. cnjs 太局限了。。\r\n>  \r\n> —\r\n> Reply to this email directly or view it on GitHub (https://github.com/cnpm/cnpmjs.org/pull/306/files#r11154351).\r\n>  \r\n>  \r\n>  ","created_at":"2014-04-01T10:14:15Z","updated_at":"2014-04-01T10:14:15Z","html_url":"https://github.com/cnpm/cnpmjs.org/pull/306#discussion_r11154371","pull_request_url":"https://api.github.com/repos/cnpm/cnpmjs.org/pulls/306","_links":{"self":{"href":"https://api.github.com/repos/cnpm/cnpmjs.org/pulls/comments/11154371"},"html":{"href":"https://github.com/cnpm/cnpmjs.org/pull/306#discussion_r11154371"},"pull_request":{"href":"https://api.github.com/repos/cnpm/cnpmjs.org/pulls/306"}}}}