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

A round of edits #1

Open
wants to merge 14 commits into
base: whitespace
Choose a base branch
from
Open

A round of edits #1

wants to merge 14 commits into from

Conversation

ceastlund
Copy link
Owner

Mostly just rewording. See how it looks to you. Also, the references section could use some work to look more like a bibliography.

Copy link

@pitag-ha pitag-ha left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice, thanks a lot for the in-depth go-through! It's way clearer this way, and you've also caught a couple of things that were misleading to wrong before.

proposal/proposal.tex Outdated Show resolved Hide resolved
proposal/proposal.tex Outdated Show resolved Hide resolved
proposal/proposal.tex Outdated Show resolved Hide resolved
proposal/proposal.tex Outdated Show resolved Hide resolved
proposal/proposal.tex Outdated Show resolved Hide resolved
proposal/proposal.tex Outdated Show resolved Hide resolved
Copy link

@panglesd panglesd left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Some query-replace that could be done (I don't know how to do them in the PR, and opening another would yield conflicts...):

  • $ppxlib$ -> \texttt{ppxlib}
  • $metaquot$ -> \texttt{metaquot}
  • $opam$ -> \texttt{opam}
  • $ocaml\mbox{-}migrate\mbox{-}parsetree$ -> \texttt{ocaml-migrate-parsetree}
  • $ppx \_ deriving$ -> \texttt{ppx_deriving}
  • $merlin$ -> \texttt{merlin}
  • $Stdlib$ -> \texttt{Stdlib}
  • $compare$ -> \texttt{compare}
  • $Ast\_builder$ -> \texttt{Ast_builder}
  • $Ast\_pattern$ -> \texttt{Ast_pattern}
  • $ppx\_universe$ -> \texttt{ppx_universe}
  • $ppxlib.0.26.0 $ -> \texttt{ppxlib.0.26.0}
  • $ocaml.4.02.00$ -> \texttt{ocaml.4.02.00}

@ceastlund
Copy link
Owner Author

The convention I tried to go with was math-italics for names of things, and fixed-width for literal OCaml code. But I'm open to whatever other convention people want to go with.

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.

3 participants