-
Notifications
You must be signed in to change notification settings - Fork 5
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
Change #1
base: master
Are you sure you want to change the base?
Change #1
Conversation
…uple struct to a c-type one
Hi there and thanks for putting this together. I notice that there's also a fair amount of restructuring in this commit. Is that intentional, and if so is there any particular reason to favor the new structure? |
I'm curious about this as well as I need support for the latest pandoc-types. Pinging @fizzbucket. @elliottslaughter, are you primarily referring to the removal of the |
Right, there seem to be four distinct kinds of changes in this commit:
I'm happy with all of these except (3), which just seems like an anti-pattern to me. Maybe I'm wrong though. Or maybe we could I don't have time at the moment, but maybe in the next couple of evenings I can try to find time to split out (1), (2) and (4) so they can be committed into the repo separately, which would at least unblock users who want to use newer Pandoc Types versions. And then we can have the API discussion separately. |
I was looking at this and realized that the For now I've just bumped the API version to 1.20. This is sufficient to make my local tests pass. I would be open to further improvements to require less hard-coding, but I'd like to know how we're going to solve the issue with users needing to know what API version to create documents for. @dkasak if you pull you should be able to use this with the latest pandoc-types. |
Ok, I think I've pulled in all of (1), (2) and (4), and have made a new release on crates.io. Please update your |
This pull request updates the pandoc struct from a tuple struct to a c-type struct so that a hardcoded api version is no longer required. This will let the crate work with the latest versions of Pandoc. It also adds some more testcases and makes minor changes to 2018 edition idioms.