Skip to content

fix(ci): create tag before git-cliff in workflow_dispatch releases#47

Merged
jdsika merged 1 commit into
fix/credential-status-did-compliancefrom
fix/release-changelog
Apr 23, 2026
Merged

fix(ci): create tag before git-cliff in workflow_dispatch releases#47
jdsika merged 1 commit into
fix/credential-status-did-compliancefrom
fix/release-changelog

Conversation

@jdsika
Copy link
Copy Markdown
Collaborator

@jdsika jdsika commented Apr 23, 2026

Summary

Port the release changelog fix from harbour-credentials to simpulse-id-credentials.

Changes

  • Add "Ensure tag exists for changelog generation" step to cd-release.yml before the git-cliff action
  • Remove continue-on-error: true from the changelog step so failures are visible

Context

When the release workflow is triggered via workflow_dispatch, the target tag does not yet exist in git history. git-cliff --latest then picks up the previous tag and generates an incorrect changelog (or fails silently due to continue-on-error).

This is the same bug that was fixed in harbour-credentials (2c73d30).

Testing

  • Workflow YAML is valid
  • Tag push trigger still works (tag already exists → no-op)
  • workflow_dispatch trigger creates lightweight tag before changelog generation

Related

  • harbour-credentials fix: 2c73d30 (fix(ci): create tag before git-cliff in workflow_dispatch releases)

When the release workflow is triggered via workflow_dispatch, the target tag
does not yet exist in git history. git-cliff --latest picks up the previous
tag instead of generating notes for the new release.

Fix: create a lightweight local tag before running git-cliff so --latest
resolves to the correct range.

Also removes continue-on-error from the changelog step so failures are
surfaced instead of silently swallowed.

Signed-off-by: Carlo van Driesten <carlo.van-driesten@bmw.de>
@jdsika jdsika self-assigned this Apr 23, 2026
@jdsika jdsika changed the base branch from main to fix/credential-status-did-compliance April 23, 2026 20:00
@jdsika jdsika merged commit f2c12da into fix/credential-status-did-compliance Apr 23, 2026
17 checks passed
jdsika added a commit that referenced this pull request May 7, 2026
* fix(examples)!: migrate credentialStatus to DID-based CRSet model

BREAKING CHANGE: credentialStatus entries use the new statusServiceOperator
+ statusIndex model instead of the old non-compliant DID path id notation.

- Migrate all 5 SimpulseID credential examples to new format
- Update test_examples_integrity.py: check statusServiceOperator and statusIndex
- Update docs/credentials/index.md credential skeleton
- Pin harbour-credentials submodule to fix/credential-status-did-compliance

Refs: reachhaven/harbour-credentials#6
Signed-off-by: Carlo van Driesten <carlo.van-driesten@bmw.de>

* fix(ci): create tag before git-cliff in workflow_dispatch releases (#47)

When the release workflow is triggered via workflow_dispatch, the target tag
does not yet exist in git history. git-cliff --latest picks up the previous
tag instead of generating notes for the new release.

Fix: create a lightweight local tag before running git-cliff so --latest
resolves to the correct range.

Also removes continue-on-error from the changelog step so failures are
surfaced instead of silently swallowed.

Signed-off-by: Carlo van Driesten <carlo.van-driesten@bmw.de>

* chore(deps): pin harbour-credentials submodule to v1.1.0 release

The PR previously pinned to a feature branch commit (dd572d6) on
fix/credential-status-did-compliance. Update to the v1.1.0 release tag
(b89ddae) which includes the same CRSet model fix via squash-merge,
plus CI improvements and the TypeScript pipeline.

Refs: reachhaven/harbour-credentials#6
Signed-off-by: Carlo van Driesten <carlo.van-driesten@bmw.de>

---------

Signed-off-by: Carlo van Driesten <carlo.van-driesten@bmw.de>
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.

1 participant