Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

docs: add resource page #364

Merged
merged 7 commits into from
Feb 13, 2017
Merged

docs: add resource page #364

merged 7 commits into from
Feb 13, 2017

Conversation

popomore
Copy link
Member

@popomore popomore commented Feb 13, 2017

Checklist
  • npm test passes
  • tests and/or benchmarks are included
  • documentation is changed or added
  • commit message follows commit guidelines
Affected core subsystem(s)
Description of change

Closes #287

@mention-bot
Copy link

@popomore, thanks for your PR! By analyzing the history of the files in this pull request, we identified @dead-horse and @atian25 to be potential reviewers.

@popomore popomore added this to the 1.0.0 milestone Feb 13, 2017
title: 资源
---

## 渠道
Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

加了个渠道,现在好像没别的

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

渠道? 指啥? 有点怪怪的

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

叫什么好呢,可以在哪里找到我们,像微博,twitter,虽然我们现在都没有。

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

感觉是不是要有个讨论组之类,邮箱或者issue

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

交流通道 ?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

现在东西比较少,比较难归类,我还是放在社区文章里吧

@codecov
Copy link

codecov bot commented Feb 13, 2017

Codecov Report

Merging #364 into master will not change coverage.
The diff coverage is n/a.

@@          Coverage Diff          @@
##           master   #364   +/-   ##
=====================================
  Coverage     100%   100%           
=====================================
  Files          28     28           
  Lines         636    636           
=====================================
  Hits          636    636

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update c52cbbf...bf6b4ec. Read the comment docs.

for (const [ pid, count ] of map) {
console.log('pid: %s, %s', pid, count);
}
map.size.should.equal(4);
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

assert

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

这个一起改吧

Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

ok

@popomore popomore mentioned this pull request Feb 13, 2017
4 tasks
@fengmk2 fengmk2 merged commit 0bbfa7b into master Feb 13, 2017
@fengmk2 fengmk2 deleted the doc-resource branch February 13, 2017 09:51
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

文档添加 resources 页面
5 participants