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

Update README.md #5

Closed
wants to merge 1 commit into from
Closed

Update README.md #5

wants to merge 1 commit into from

Conversation

yihuishou
Copy link

Add Missed Something…

Add Missed Something…
@@ -30,6 +30,7 @@ theme: iris
The following Hexo plugins is necessary:

* [hexo-renderer-ejs](https://www.npmjs.com/package/hexo-renderer-ejs)
* [hexo-generator-feed](https://www.npmjs.com/package/hexo-generator-feed)
Copy link
Owner

@giuem giuem May 15, 2019

Choose a reason for hiding this comment

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

Hmmm, actually this plugin is not necessary if you comment out head.rss field in theme config.

Copy link
Author

Choose a reason for hiding this comment

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

Emmm… 如果这个非必要的话,那么默认的 _config.example.yml 文件中为啥没被注释掉呢?
我觉得对于一个新安装的 Hexo 然后使用了这个主题,按照示例说明,使用了给出的默认配置…
最后报错无法运行,这个用户体验不是很好的说

@yihuishou yihuishou closed this May 15, 2019
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.

2 participants