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 dune files for opam-repo checks #2263

Merged
merged 2 commits into from
Mar 6, 2023
Merged

Conversation

kit-ty-kate
Copy link
Contributor

@@ -1,4 +1,7 @@
(test
(name test_unit)
(deps
../passing/tests/partial.ml
../passing/tests/format_invalid_files_with_locations.ml)
Copy link
Collaborator

Choose a reason for hiding this comment

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

There's a read to ../../lib/Fmt_ast.ml in Test_translation_unit but I think you can remove the read call.

Copy link
Collaborator

Choose a reason for hiding this comment

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

This patch is minimal to please the CI of ocaml/opam-repository#23407 so I will merge it as it is.

@gpetiot gpetiot added the no changelog set this to bypass the CI check for changelog entries label Mar 6, 2023
@gpetiot gpetiot changed the title Fix dune files Fix dune files for opam-repo checks Mar 6, 2023
@gpetiot gpetiot merged commit 582750f into ocaml-ppx:main Mar 6, 2023
@gpetiot gpetiot mentioned this pull request Mar 6, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
no changelog set this to bypass the CI check for changelog entries
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants