Skip to content

use commits from haddock3 repo as version#8

Merged
rvhonorato merged 1 commit into
mainfrom
7-allow-pulling-an-unreleased-haddock3-versioncommit
Jul 17, 2026
Merged

use commits from haddock3 repo as version#8
rvhonorato merged 1 commit into
mainfrom
7-allow-pulling-an-unreleased-haddock3-versioncommit

Conversation

@rvhonorato

Copy link
Copy Markdown
Member

as mentioned here: #5 (comment), there is an use-case in which people might want to use the benchmarking pipeline on a version of haddock3 that has not yet been published to pypi.

this PR adds the possibility of pulling either a version from pypi OR a commit hash from the repository:

# > use a commit (must exist on haddocking/haddock3)
HADDOCK3_VERSION="6261effa28a673d7f8513ef8f5a4c0f231f24ebc"
# > use pypi version (must exist on pypi)
# HADDOCK3_VERSION=2026.7.0

Important

Note that I deliberately set it as needing to take a commit from the remote, not local. This is so that we can keep some reproducibility even for unreleased sources - if someone is using a commit hash from their local copy, there is no way someone else can reproduce it. If its a commit hash from the remote, even if it never gets pushed into the main branch, it will remain on the repository history and can thus be reproduced. But keep in mind that over time the commits might not be accessible, so PREFER using pypi versions.

@rvhonorato
rvhonorato requested a review from amjjbonvin July 17, 2026 07:56
@rvhonorato rvhonorato self-assigned this Jul 17, 2026
@rvhonorato rvhonorato linked an issue Jul 17, 2026 that may be closed by this pull request

@amjjbonvin amjjbonvin left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Nice

@rvhonorato
rvhonorato merged commit c02db06 into main Jul 17, 2026
@rvhonorato
rvhonorato deleted the 7-allow-pulling-an-unreleased-haddock3-versioncommit branch July 17, 2026 08:08
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.

allow pulling an unreleased haddock3 version/commit

2 participants