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

rewrite tools/pandoc-*.hs as lua scripts #670

Closed
simonmichael opened this Issue Dec 8, 2017 · 5 comments

Comments

Projects
None yet
2 participants
@simonmichael
Owner

simonmichael commented Dec 8, 2017

Shake.hs will soon depend on pandoc 2, which has built in support for filters written in lua. If we rewrite our haskell pandoc filters in lua (tools/pandoc-*hs, just the ones being used by Shake.hs), we'll remove another source of complexity and fragility from the doc toolchain. Need someone to help with this.

@simonmichael

This comment has been minimized.

Show comment
Hide comment
@simonmichael

simonmichael Dec 9, 2017

Owner

Getting-started ideas:

Pick one filter, eg https://github.com/simonmichael/hledger/blob/master/tools/pandoc-drop-html-blocks.hs
Get that filter working on command line, cf linked docs
Get a similar lua filter working on command line
Modify Shake.hs to use the lua filter instead of the haskell one
Watch compilation errors: stack exec --package base-prelude -- ghcid Shake.hs
Repeat

Owner

simonmichael commented Dec 9, 2017

Getting-started ideas:

Pick one filter, eg https://github.com/simonmichael/hledger/blob/master/tools/pandoc-drop-html-blocks.hs
Get that filter working on command line, cf linked docs
Get a similar lua filter working on command line
Modify Shake.hs to use the lua filter instead of the haskell one
Watch compilation errors: stack exec --package base-prelude -- ghcid Shake.hs
Repeat

@simonmichael simonmichael added the easy? label Dec 9, 2017

@ehildenb

This comment has been minimized.

Show comment
Hide comment
@ehildenb

ehildenb Apr 15, 2018

Contributor

Anyone working on this? I can give a try.

Contributor

ehildenb commented Apr 15, 2018

Anyone working on this? I can give a try.

@simonmichael

This comment has been minimized.

Show comment
Hide comment
@simonmichael

simonmichael Apr 15, 2018

Owner

Please do!

Owner

simonmichael commented Apr 15, 2018

Please do!

This was referenced Apr 21, 2018

@ehildenb

This comment has been minimized.

Show comment
Hide comment
@ehildenb

ehildenb Apr 26, 2018

Contributor

@simonmichael I think you can close due to #741.

Contributor

ehildenb commented Apr 26, 2018

@simonmichael I think you can close due to #741.

@simonmichael

This comment has been minimized.

Show comment
Hide comment
@simonmichael

simonmichael Apr 26, 2018

Owner

True, and PS it's working great! No more waiting for pandoc filters to compile, and one less reason to need to wait for pandoc to compile.

@ehildenb, if you have any time to look at replacing hakyll-std with just pandoc, that would be quite useful too. If not, I'll try it one of these days.

Owner

simonmichael commented Apr 26, 2018

True, and PS it's working great! No more waiting for pandoc filters to compile, and one less reason to need to wait for pandoc to compile.

@ehildenb, if you have any time to look at replacing hakyll-std with just pandoc, that would be quite useful too. If not, I'll try it one of these days.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment