-
Notifications
You must be signed in to change notification settings - Fork 101
feat: add the latex serializer #393
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
base: main
Are you sure you want to change the base?
Conversation
Signed-off-by: Peter Staar <[email protected]>
|
✅ DCO Check Passed Thanks @PeterStaar-IBM, all your commits are properly signed off. 🎉 |
Merge ProtectionsYour pull request matches the following merge protections and will not be merged until they are valid. 🔴 Require two reviewer for test updatesThis rule is failing.When test data is updated, we require two reviewers
🟢 Enforce conventional commitWonderful, this rule succeeded.Make sure that we follow https://www.conventionalcommits.org/en/v1.0.0/
|
Signed-off-by: Peter Staar <[email protected]>
Codecov Report❌ Patch coverage is
📢 Thoughts on this report? Let us know! |
Signed-off-by: Peter Staar <[email protected]>
Signed-off-by: Peter Staar <[email protected]>
| page_break_command: Optional[str] = None | ||
|
|
||
| # Escape LaTeX special characters in text | ||
| escape_latex: bool = True |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Would be great to have some test coverage for all introduced params (I think e.g. these two are not covered).
* working on adding complex table support Signed-off-by: Peter Staar <[email protected]> * refactored the code Signed-off-by: Peter Staar <[email protected]> * updated the latex gt Signed-off-by: Peter Staar <[email protected]> --------- Signed-off-by: Peter Staar <[email protected]>
d533e71 to
1bed3f6
Compare
Signed-off-by: Peter Staar <[email protected]>
Let's first merge this: #402