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: wrong logbook creation #4382
Conversation
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.
@llcc Confirmed this fixes the first two bugs/enhancements! 👍 🚢 There's some minor feedback in marker.cljs to think about
… into fix-wrong-logbook-creation
@llcc It seems that bumping mldoc made the CI tests fail, I've updated yarn.lock in the master branch. Rebase on master should fix it. |
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.
LGTM
4e4b3c7
to
f90fca1
Compare
No need to have set-marker impl copied with with-redefs. Move tests to its corresponding test ns
…ook-creation Dev: Rewrite tests from #4382 to use with-redefs
DOING/NOW
marker exists in center of block. close LOGBOOK is generated in WARNING Admonition #3746, only show the first property, caused by the logbook drawer produced by a block have the "DOING" string #4381