Skip to content

add ahrefs config in tests #2695

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

Merged
merged 1 commit into from
Apr 18, 2025

Conversation

EmileTrotignon
Copy link
Collaborator

This does not add a profile, it just tests the config used by ahrefs.

@EmileTrotignon
Copy link
Collaborator Author

I am merging this for convenience of testing other PRs, if there is a reason not to have it I am okay with this being deleted later.

I think it has value for the community because testing a new option combination is going to make the coverage of the testsuite better, but it will also make the test run a little slower, which is a reason not to do it.

@EmileTrotignon EmileTrotignon added no changelog set this to bypass the CI check for changelog entries review-merged This PR was merged, but would benefit from reviewing after the fact labels Apr 18, 2025
@EmileTrotignon EmileTrotignon merged commit 9ae5986 into ocaml-ppx:main Apr 18, 2025
10 of 12 checks passed
davesnx pushed a commit to ocaml-mlx/ocamlformat-mlx that referenced this pull request May 6, 2025
davesnx pushed a commit to ocaml-mlx/ocamlformat-mlx that referenced this pull request May 6, 2025
@Julow Julow removed the review-merged This PR was merged, but would benefit from reviewing after the fact label May 7, 2025
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.

2 participants