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

HTML and latex generation are not checked by dune build @install #3416

Open
emillon opened this issue Dec 3, 2020 · 1 comment
Open

HTML and latex generation are not checked by dune build @install #3416

emillon opened this issue Dec 3, 2020 · 1 comment

Comments

@emillon
Copy link
Contributor

emillon commented Dec 3, 2020

In #3406, some blocks look like this:

```sh skip
>>> (ls .)
Could not find matching handler: ls
```

This triggers (and should) an error in mdx, but this only happens in @html, not in @runtest. As a result, it's missed by ocaml-ci which runs @install for monorepo builds. As a workaround, this alias (or @site? what about @latex?) can be added to @install.

emillon added a commit to emillon/book that referenced this issue Dec 3, 2020
emillon added a commit to emillon/book that referenced this issue Dec 3, 2020
@yminsky
Copy link
Member

yminsky commented Dec 5, 2020

I'm surely missing something, but what's wrong with the listed block? Why should it error out at all?

emillon added a commit to emillon/book that referenced this issue Dec 23, 2020
emillon added a commit to emillon/book that referenced this issue Jul 5, 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 a pull request may close this issue.

2 participants