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.

Sign up for GitHub

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

Jump to bottom

Fix Sexplib parse fails #67

Merged

moyodiallo merged 1 commit into ocurrent:master from moyodiallo:fix-parse

Mar 28, 2024

Collaborator

moyodiallo commented Mar 28, 2024

Fixes #66


          Fix Sexplib parse fails

3dc1026

Fixes ocurrent#66

moyodiallo merged commit 14f790e into ocurrent:master

1 check passed

moyodiallo deleted the fix-parse branch

March 28, 2024 17:12

moyodiallo mentioned this pull request

[new release] opam-dune-lint (0.6) ocaml/opam-repository#25698

Merged

mseri pushed a commit to ocaml/opam-repository that referenced this pull request


          [new release] opam-dune-lint (0.6)

f7a1afe

CHANGES:

- Fix the issue ocurrent/opam-dune-lint#68, with Sexp format dune parse quoted string by escaping "%{" to become "\%{" that OpamFile module can't parse. So we switch to Csexp format which doesn't change quoted string (@moyodiallo ocurrent/opam-dune-lint#69).

- Fix the issue ocurrent/opam-dune-lint#66, when the content of dune-project file ends up with a comment, the Sexplib parse fails because the last `)` falls into the comment (@moyodiallo ocurrent/opam-dune-lint#67).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet