Skip to content

Apply merl-an on main branch #1896

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
Merged
merged 5 commits into from
Mar 4, 2025
Merged

Apply merl-an on main branch #1896

merged 5 commits into from
Mar 4, 2025

Conversation

xvw
Copy link
Collaborator

@xvw xvw commented Feb 18, 2025 •
edited
Loading

Awaiting by #1905

@xvw xvw added the no changelog Turn off CI changelog-check label Feb 18, 2025
@xvw xvw force-pushed the restore-mer-lan-in-the-CI branch from a65d00d to fe3181d Compare February 18, 2025 13:08
@xvw xvw self-assigned this Feb 18, 2025
@xvw xvw force-pushed the restore-mer-lan-in-the-CI branch 27 times, most recently from 5962ede to 78311a3 Compare March 4, 2025 04:28
@xvw xvw mentioned this pull request Mar 4, 2025
Merged
@xvw xvw changed the title Draft: Apply merl-an on main branch Apply merl-an on main branch Mar 4, 2025
@xvw xvw force-pushed the restore-mer-lan-in-the-CI branch 2 times, most recently from 559b93c to f3a38a0 Compare March 4, 2025 05:18
@xvw xvw force-pushed the restore-mer-lan-in-the-CI branch from 2ea4a0a to 8c24ac4 Compare March 4, 2025 10:17
@github-actions GitHub Actions
Copy link

github-actions bot commented Mar 4, 2025

This PR changes the response of some of the ocamlmerlin queries, that were run and analyzed by the Merlin Fuzzy CI. The change is not considered a regression, the analysis of this PR has been approved in its following state:

  • URL to download the generated data sets and their diffs between PR base branch and merge branch (at the moment of approval): https://github.com/ocaml/merlin/actions/runs/13651395882
  • 256-sha of full responses diff: 65fe835361e806ec6d0a7c392b030ada7d1b9cff6abe61bd08bd25bcfd7245c6

voodoos
voodoos approved these changes Mar 4, 2025
View reviewed changes
Copy link
Collaborator

@voodoos voodoos left a comment

Choose a reason for hiding this comment

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

Thanks a lot @xvw , let's see how it goes.

@voodoos voodoos merged commit 6d95435 into ocaml:main Mar 4, 2025
14 of 15 checks passed
@xvw xvw deleted the restore-mer-lan-in-the-CI branch March 4, 2025 10:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@voodoos voodoos voodoos approved these changes

Assignees

@xvw xvw

Labels
fuzzy-diff-looks-good no changelog Turn off CI changelog-check
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@xvw @voodoos