Skip to content

Fix documentation strings#30

Merged
mato merged 1 commit into
mirage:masterfrom
hannesm:minor
Oct 27, 2020
Merged

Fix documentation strings#30
mato merged 1 commit into
mirage:masterfrom
hannesm:minor

Conversation

@hannesm
Copy link
Copy Markdown
Member

@hannesm hannesm commented Oct 26, 2020

The documentation comments in OCaml are opened with "(**" and closed with ")".
All characters in between the open and close markers are interpret as
documentation. This lead to '
' being part of the docstring, see:
https://mirage.github.io/mirage-xen/mirage-xen/OS/Memory/index.html#type-stat

The documentation comments in OCaml are opened with "(**" and closed with "*)".
All characters in between the open and close markers are interpret as
documentation. This lead to '*' being part of the docstring, see:
https://mirage.github.io/mirage-xen/mirage-xen/OS/Memory/index.html#type-stat
@mato
Copy link
Copy Markdown
Contributor

mato commented Oct 27, 2020

Thanks. What's the go-to quick way of previewing the documentation locally when making changes?

@mato mato merged commit 8d782cd into mirage:master Oct 27, 2020
@hannesm
Copy link
Copy Markdown
Member Author

hannesm commented Oct 27, 2020

@mato dune build @doc uses odoc (you may need to opam install odoc) to generate html output (located in _build/default/_doc/_html/).

@hannesm hannesm deleted the minor branch October 27, 2020 09:55
hannesm added a commit to hannesm/opam-repository that referenced this pull request Nov 18, 2021
CHANGES:

* Fix OCaml documentation strings (mirage/mirage-xen#30 @hannesm)
* Add caml_get_wall_clock to retrieve the wall clock as int64 (mirage/mirage-xen#31 @hannesm)
* Don't require opam for the bindings compilation (mirage/mirage-xen#32 @sternenseemann)
* Remove checksum_stubs and alloc_pages_stubs (available in mirage-tcpip
  and io-page), to keep in sync with mirage-solo5 (mirage/mirage-xen#33 @dinosaure, adapted
  by @hannesm to mirage-xen)
arbipher pushed a commit to arbipher/opam-repository that referenced this pull request Nov 21, 2021
CHANGES:

* Fix OCaml documentation strings (mirage/mirage-xen#30 @hannesm)
* Add caml_get_wall_clock to retrieve the wall clock as int64 (mirage/mirage-xen#31 @hannesm)
* Don't require opam for the bindings compilation (mirage/mirage-xen#32 @sternenseemann)
* Remove checksum_stubs and alloc_pages_stubs (available in mirage-tcpip
  and io-page), to keep in sync with mirage-solo5 (mirage/mirage-xen#33 @dinosaure, adapted
  by @hannesm to mirage-xen)
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.

2 participants