Skip to content

Revert "Fix release location calculation (#205)"#211

Merged
Leonidas-from-XIV merged 1 commit intoocaml-dune:mainfrom
Leonidas-from-XIV:revert-release-location
Mar 4, 2026
Merged

Revert "Fix release location calculation (#205)"#211
Leonidas-from-XIV merged 1 commit intoocaml-dune:mainfrom
Leonidas-from-XIV:revert-release-location

Conversation

@Leonidas-from-XIV
Copy link
Contributor

The workflows fail because they can't connect via SSH to the server anymore. This was confusing why it would happen, so I talked to @mtelvers who suggested reverting this.

This reverts commit 493df29.

I don't really know why this would make a difference since the only thing it changes is instead of writing to /dune/2026-03-04 is to write to /dune/3.20.1 but might as well try to undo it in an attempt to debug the issue.

This reverts commit 493df29.

Signed-off-by: Marek Kubica <marek@tarides.com>
Copy link
Contributor

@mtelvers mtelvers left a comment

Choose a reason for hiding this comment

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

As it was the only change to the repo, the logical step is to revert and see if the problem clears.

@Leonidas-from-XIV Leonidas-from-XIV force-pushed the revert-release-location branch from 41e811e to 0382a03 Compare March 4, 2026 09:11
@Leonidas-from-XIV Leonidas-from-XIV merged commit a34f2a3 into ocaml-dune:main Mar 4, 2026
3 checks passed
@Leonidas-from-XIV Leonidas-from-XIV deleted the revert-release-location branch March 4, 2026 09:43
Leonidas-from-XIV added a commit to Leonidas-from-XIV/dune-binary-distribution that referenced this pull request Mar 4, 2026
Leonidas-from-XIV added a commit to Leonidas-from-XIV/dune-binary-distribution that referenced this pull request Mar 4, 2026
…ne#211)

This reverts commit a34f2a3.

Signed-off-by: Marek Kubica <marek@tarides.com>
Leonidas-from-XIV added a commit that referenced this pull request Mar 4, 2026
Signed-off-by: Marek Kubica <marek@tarides.com>
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.

2 participants