Skip to content

Conversation

@gasche
Copy link
Contributor

@gasche gasche commented Oct 10, 2025

No description provided.

@gasche gasche mentioned this pull request Oct 10, 2025
@gasche
Copy link
Contributor Author

gasche commented Oct 10, 2025

While I am at it, I updated the test matrix to remove versions strictly older than 4.14, and add later versions. (I also added 5.4, which should fail.)

@gasche
Copy link
Contributor Author

gasche commented Oct 10, 2025

The test results are as I expected: it breaks on 5.4 and works on older versions (and now the CI runs). I am merging the present PR so that #100 can be tested properly.

@gasche gasche merged commit 375d04d into ocaml-ppx:master Oct 10, 2025
5 of 7 checks passed
ejgallego added a commit to ejgallego/ppx_import that referenced this pull request Oct 13, 2025
This is a 1.x version of Gabriel's ocaml-ppx#101 for main.
ejgallego added a commit to ejgallego/ppx_import that referenced this pull request Oct 13, 2025
This is a 1.x version of Gabriel's ocaml-ppx#101 for main.
ejgallego added a commit to ejgallego/ppx_import that referenced this pull request Oct 13, 2025
This is a 1.x version of Gabriel's ocaml-ppx#101 for main.
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.

1 participant