Sponsored Content
Skip to content

ci: Add missing commit hash to Conan recipe version - #6256

Merged
bthomee merged 1 commit into
developfrom
bthomee/sha
Jan 22, 2026
Merged

ci: Add missing commit hash to Conan recipe version#6256
bthomee merged 1 commit into
developfrom
bthomee/sha

Conversation

@bthomee

@bthomee bthomee commented Jan 21, 2026

Copy link
Copy Markdown
Collaborator

High Level Overview of Change

This change adds the commit hash as suffix to the Conan recipe version.

Context of Change

During several iterations of development the commit hash was supposed to be moved into the run: statement, but it slipped through the cracks and did not get added.

Type of Change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)
  • Refactor (non-breaking change that only restructures code)
  • Performance (increase or change in throughput and/or latency)
  • Tests (you added tests for code that already exists, or your new feature included in this PR)
  • Documentation update
  • Chore (no impact to binary, e.g. .gitignore, formatting, dropping support for older tooling)
  • Release

@bthomee
bthomee requested a review from mathbunnyru January 22, 2026 00:01
@legleux
legleux self-requested a review January 22, 2026 00:15
@bthomee
bthomee merged commit 4fd4e93 into develop Jan 22, 2026
1 check passed
@bthomee
bthomee deleted the bthomee/sha branch January 22, 2026 00:17
@mvadari mvadari added this to the 3.2.0 milestone May 20, 2026
beartec-jpg pushed a commit to beartec-jpg/FalconLedger that referenced this pull request Jun 1, 2026
During several iterations of development of XRPLF#6235, the commit hash was supposed to be moved into the `run:` statement, but it slipped through the cracks and did not get added. This change adds the commit hash as suffix to the Conan recipe version.
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.

3 participants