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

Modification to work with Hexo3 #4

Merged
merged 3 commits into from Mar 21, 2015
Merged

Modification to work with Hexo3 #4

merged 3 commits into from Mar 21, 2015

Conversation

ntalbs
Copy link
Contributor

@ntalbs ntalbs commented Mar 20, 2015

Current hexo-math doesn't work with Hexo3. I modified some part to use correct module (newly introduced in Hexo 3). I couldn't correct themeDir in Command.js. I just hard coded this part to avoid error. I hope you merge this pull request and correct the themeDir in Command.js that hexo-math would work well with Hexo3.

`hexo.util` and `hexo-file` is undefined.
Update to use hexo-util, hexo-fs module.
hexo is undefined here. Update to use `hexo-fs` module. I coundn't correct hexo.theme_dir since I don't have sufficient knowledge about node.js, hexo plugin, etc. I hope you can correct this part.
comment out first line because it's not used in this file.
update util, file to require proper module.
@ntalbs
Copy link
Contributor Author

ntalbs commented Mar 20, 2015

And the documentation should be updated, too. {% math-block %} ... {% endmath-block %} cause error in Hexo3. I tried {% block math %} ... {% endblock %} and it worked well.

@akfish
Copy link
Member

akfish commented Mar 20, 2015

Thanks for your contribution. I'll review these changes and merge this PL soon.

@akfish akfish self-assigned this Mar 21, 2015
akfish added a commit that referenced this pull request Mar 21, 2015
akfish added a commit that referenced this pull request Mar 21, 2015
@akfish akfish merged commit 8689e3d into hexojs:master Mar 21, 2015
@akfish
Copy link
Member

akfish commented Mar 21, 2015

Fixed and published as version 1.0.5.

Block tag renamed to {% math_block%} and {% endmath_block %}.

Thanks for your contribution again.

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.

None yet

2 participants