Skip to content

GHA: Harden the script against bugs in actions/runner-images in case GITHUB_WORKSPACE gets moved from D: to C: in the Windows image#6553

Merged
kit-ty-kate merged 1 commit intoocaml:masterfrom
kit-ty-kate:gha-move-opam-repository-location
Jun 17, 2025
Merged

Conversation

@kit-ty-kate
Copy link
Copy Markdown
Member

@kit-ty-kate kit-ty-kate added this to the 2.4.0~beta1 milestone Jun 16, 2025
@kit-ty-kate kit-ty-kate force-pushed the gha-move-opam-repository-location branch from 85787fa to 74aa556 Compare June 16, 2025 19:28
@kit-ty-kate kit-ty-kate changed the title GHA: move the location of opam-repository to a relative directory consistent accross platforms and potential bugs in actions/runner-images GHA: move the location of opam-repository to a relative directory consistent accross platforms and resistant to bugs in actions/runner-images Jun 16, 2025
@kit-ty-kate kit-ty-kate force-pushed the gha-move-opam-repository-location branch 2 times, most recently from 972d138 to 58112de Compare June 17, 2025 00:34
@kit-ty-kate kit-ty-kate changed the title GHA: move the location of opam-repository to a relative directory consistent accross platforms and resistant to bugs in actions/runner-images GHA: Harden the script against bugs in actions/runner-images in case GITHUB_WORKSPACE gets moved from D: to C: in the Windows image Jun 17, 2025
@kit-ty-kate kit-ty-kate force-pushed the gha-move-opam-repository-location branch from 03091a7 to 265545d Compare June 17, 2025 12:14
@kit-ty-kate kit-ty-kate requested a review from rjbou June 17, 2025 12:14
Copy link
Copy Markdown
Collaborator

@rjbou rjbou left a comment

Choose a reason for hiding this comment

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

thanks!

@kit-ty-kate kit-ty-kate merged commit 2666453 into ocaml:master Jun 17, 2025
85 of 86 checks passed
@kit-ty-kate kit-ty-kate deleted the gha-move-opam-repository-location branch June 17, 2025 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants