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

add rasa blog #142

Merged
merged 5 commits into from
Jul 24, 2019
Merged

add rasa blog #142

merged 5 commits into from
Jul 24, 2019

Conversation

qhduan
Copy link
Contributor

@qhduan qhduan commented Jul 23, 2019

Fixes # .

Make sure these boxes are checked before creating your pull request -- thank you!

  • Add Blog Header, including title, author, date.
    e.g.
---
 
 title: "'Score Your Face Photo' a ML&Wechaty practice"
 date: 2017-09-18 09:00 +0800
 author: huyingxi
 ---
 
 > Author: [@huyingxi](https://github.com/huyingxi/wechaty_selfie) enjoying ML&Wechaty at BUPT
  • Keep all filenames & url as lowercase, and use - to connect words instead of space. e.g. 2017-10-06-wechat-pc-impactor instead of 2017-10-06-WeChat PC Impactor,
  • Embed the photo & video before publishing, save all external file to the blog /download/2018 directory.
  • Add <!--more--> section to just show the abstract in blog homepage.

See more

@CLAassistant
Copy link

CLAassistant commented Jul 23, 2019

CLA assistant check
All committers have signed the CLA.

@qhduan
Copy link
Contributor Author

qhduan commented Jul 23, 2019

@lijiarui @huan

Copy link
Member

@huan huan left a comment

Choose a reason for hiding this comment

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

Dear @qhduan , thank you for the great blog post, It is very valuable of sharing your thoughts to the community!

In order to publish your blog post, we need to make sure the PR can pass the CI test.

Currently, there are some problems from the CI reported that:

docs/_posts/2019-07-23-rasa.md: 100: MD029/ol-prefix Ordered list item prefix [Expected: 1; Actual: 3; Style: 1/2/3]
docs/_posts/2019-07-23-rasa.md: 106: MD029/ol-prefix Ordered list item prefix [Expected: 2; Actual: 4; Style: 1/2/3]
docs/_posts/2019-07-23-rasa.md: 108: MD029/ol-prefix Ordered list item prefix [Expected: 3; Actual: 5; Style: 1/2/3]
docs/_posts/2019-07-23-rasa.md: 110: MD029/ol-prefix Ordered list item prefix [Expected: 4; Actual: 6; Style: 1/2/3]
docs/_posts/2019-07-23-rasa.md: 153: MD012/no-multiple-blanks Multiple consecutive blank lines [Expected: 1; Actual: 2]

Please fix them (to make the markdown clean and more readable), and also sign the above CLA, then we will be able to merge this PR and publish it!

@qhduan qhduan requested a review from lijiarui as a code owner July 24, 2019 05:22
@qhduan
Copy link
Contributor Author

qhduan commented Jul 24, 2019

@huan I have fixed the PR problem

@huan
Copy link
Member

huan commented Jul 24, 2019

Great, thank you for the fixing, merged!

@huan huan merged commit 4c82471 into wechaty:master Jul 24, 2019
huan pushed a commit that referenced this pull request Jul 23, 2021
* pr for wechat puppet: customized extspam options

wechaty/puppet-wechat#142

* uploaded avatar

* updated avatar

* Delete avatar.jpeg

* upload webp avatar

* Update fi6.md
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants