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

Package mutaml.0.2 #26627

Merged
merged 1 commit into from
Sep 30, 2024
Merged

Conversation

jmid
Copy link
Contributor

@jmid jmid commented Sep 26, 2024

mutaml.0.2

A mutation tester for OCaml
Mutaml is a mutation testing tool for OCaml.
It uses a ppxlib-based preprocessor to make a series of small
breaking changes to a program's source code and then runs
the program's testsuite for each of them to catch uncaught
misbehaviour.



🐫 Pull-request generated by opam-publish v2.4.0

@jmid
Copy link
Contributor Author

jmid commented Sep 26, 2024

CI was failing linting with a missing dev-repo warning.
I've thus added a source-entry to dune-project and regenerated the opam file which should address this.

On the first CI run I also saw

  • opensuse failing to install conf-diffutils - due to sys-package-installation conflict (hence unrelated)
  • an error while running the tests with 4.14 on ppc64.

Note:

@jmid
Copy link
Contributor Author

jmid commented Sep 26, 2024

There's also a 5.2 risc-v failure apparently, which I believe is a new platform since the 0.1 release - and not one tested on ocaml-ci: https://github.com/jmid/mutaml/runs/30715018402
Again the package installs cleanly on that platform, but exhibits different failure/timeout behaviour, which causes cram test differences.

@avsm
Copy link
Member

avsm commented Sep 29, 2024

You can push constraints to the with-test stanza to stop these architectures from being tests (especially the known-based 4.14 ppc64 ones). That'll prevent these tests from popping up in other CI runs in the future.

@raphael-proust
Copy link
Collaborator

In general, tests that rely on timeout don't play very nicely with opam releases. The CI for opam-repository is often different enough from your own which makes different things fail at different times. In general I'd recommend not to have those tests in the opam package (maybe make it a different target which is not called in the opam file).

Anyway, in the meantime, these errors are acceptable so I'm merging.

@raphael-proust raphael-proust merged commit 1b4da50 into ocaml:master Sep 30, 2024
2 of 3 checks passed
@jmid jmid deleted the opam-publish-mutaml.0.2 branch September 30, 2024 16:22
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