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

Fix recent pandoc latex tables by adding calc and array (#1536, #1566) #1686

Merged
merged 1 commit into from Dec 9, 2021

Conversation

cgevans
Copy link
Contributor

@cgevans cgevans commented Dec 7, 2021

Pandoc >= 2.11.1 requires the calc package, and pandoc >= 2.11.3
requires the array package, for larger tables (taken from git blame
for pandoc's templates). This commits adds the two packages in
an order similar to that in pandoc's templates.

This should fix both #1536 and #1566.

Pandoc >= 2.11.1 requires the calc package, and pandoc >= 2.11.3
requires the array package, for larger tables (taken from git blame
for pandoc's templates).  This commits adds the two packages in
an order similar to that in pandoc's templates.
@cgevans
Copy link
Contributor Author

cgevans commented Dec 7, 2021

Also fixes #1649.

@SylvainCorlay
Copy link
Member

cc @ivanov you may be more familiar with that part of the code base - wrt pandoc.

Copy link
Member

@ivanov ivanov left a comment

Choose a reason for hiding this comment

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

Thanks for this @cgevans (and @SylvainCorlay for the ping).

This change looks good, jgm/pandoc@bd7c9eb and jgm/pandoc@7c4d7db are the relevant commits on the pandoc side.

I just kicked the tests because one of them failed due to some transient network issue.

@SylvainCorlay
Copy link
Member

Thanks @ivanov! (Btw @cgevans opened a couple of other pandoc-related PRs).

@cgevans cgevans deleted the array-and-calc branch December 18, 2021 02:31
@blink1073 blink1073 added this to the 6.4 milestone Dec 28, 2021
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

4 participants