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

fix: type of Pp.paragraphf and add changelog #19

Conversation

Alizter
Copy link
Contributor

@Alizter Alizter commented Oct 31, 2023

We define the format string internally so that it can be enforced somewhat.

@Alizter Alizter mentioned this pull request Oct 31, 2023
Copy link
Collaborator

Choose a reason for hiding this comment

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

@Alizter I think this change is enough to give you the compile time error on the ml, so you don't need the annotation in the ml once you have this change.

@rgrinberg
Copy link
Contributor

needs a rebase

We define the format string internally so that it can be enforced
somewhat.

Signed-off-by: Ali Caglayan <[email protected]>
@Alizter Alizter force-pushed the ps/branch/fix__type_of_pp_paragraphf_and_add_changelog branch from ecde6a1 to 7f5a115 Compare November 16, 2023 11:44
@Alizter
Copy link
Contributor Author

Alizter commented Nov 16, 2023

@rgrinberg done.

@rgrinberg rgrinberg merged commit fe1d7cc into ocaml-dune:master Nov 16, 2023
7 checks passed
@Alizter Alizter deleted the ps/branch/fix__type_of_pp_paragraphf_and_add_changelog branch November 16, 2023 16:56
@mbarbin mbarbin mentioned this pull request Sep 10, 2024
mbarbin added a commit to mbarbin/opam-repository-1 that referenced this pull request Sep 16, 2024
CHANGES:

- Prepare release (ocaml-dune/pp#21, @mbarbin)
  - Upgrade to `ocamlformat.0.26.2`.
  - Fmt the code
  - Add CI badge to README
  - Upgrade GitHub workflow actions dependencies (checkout@v4, setup-ocaml@v3)
  - Add more validation steps in CI
  - Add `ocamlformat` as dev-setup dependency

- Add `Pp.verbatimf`. (ocaml-dune/pp#18, @mbarbin)

- Add `Pp.paragraph` and `Pp.paragraphf` (ocaml-dune/pp#19, @Alizter)

- Remove `of_fmt` constructor. (ocaml-dune/pp#17, @Alizter)
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