rootstock icon indicating copy to clipboard operation
rootstock copied to clipboard

Adding pandoc-secnos to the build environment.

Open matthewturk opened this issue 3 years ago • 3 comments

In conjunction with https://github.com/manubot/manubot/pull/334 , this makes it possible to add section references to a manuscript. I have not updated the github hash for manubot referenced in the build environment, but will if/when that pull request is accepted.

A sample build of this is now in the yt paper build where you can see the number of sections referenced. I did note that all the sections are part of part 1, which I suspect might be an issue with the fact that I seem to be using ## as the base section number. That's probably fixable somehow on our end.

matthewturk avatar Apr 11 '22 20:04 matthewturk

AppVeyor build 1.0.281 for commit e9236385660ca66a2692244784a778cafbc5c3a9 is now complete.

Found 51 potential spelling error(s). Preview:content/02.delete-me.md:44:adipiscing content/02.delete-me.md:44:aliqua content/02.delete-me.md:44:amet content/02.delete-me.md:44:consectetur content/02.delete-me.md:44:dolore content/02.delete-me.md:44:eiusmod content/02.delete-me.md:44:elit content/02.delete-me.md:44:incididunt content/02.delete-me.md:44:ipsum content/02.delete-me.md:44:labore content/02.delete-me.md:44:Lorem content/02.delete-me.md:44:magna content/02...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot avatar Apr 11 '22 20:04 AppVeyorBot

It would make sense to merge https://github.com/manubot/manubot/pull/334 first so we can update the manubot version in the environment here

Yes, can upgrade manubot to manubot/manubot@fb3950460ea2ce2c5e17e7fcffeec1bb75b0d822

I'm in favor of adding pandoc-secnos if we think it's unlikely to break anything. For manuscripts without any @sec:... references, I'm hoping this filter will essentially do nothing.

Unfortunately, the pandoc-xnos suite hasn't been receiving frequent maintenance. So switching to pandoc-crossref is probably the right move for Manubot. However, the syntax is similar, so merging this should not interfere with switching.

dhimmel avatar Apr 14 '22 01:04 dhimmel

AppVeyor build 1.0.282 for commit 76924c16293c750cad3454edc096de39a0921688 is now complete.

Found 51 potential spelling error(s). Preview:content/02.delete-me.md:44:adipiscing content/02.delete-me.md:44:aliqua content/02.delete-me.md:44:amet content/02.delete-me.md:44:consectetur content/02.delete-me.md:44:dolore content/02.delete-me.md:44:eiusmod content/02.delete-me.md:44:elit content/02.delete-me.md:44:incididunt content/02.delete-me.md:44:ipsum content/02.delete-me.md:44:labore content/02.delete-me.md:44:Lorem content/02.delete-me.md:44:magna content/02...
The rendered manuscript from this build is temporarily available for download at:

AppVeyorBot avatar Apr 24 '22 18:04 AppVeyorBot