Skip to content

ci: add calens validation job to CI workflow - #41603

Merged
DeepDiver1975 merged 2 commits into
masterfrom
fix/calens
Jun 8, 2026
Merged

ci: add calens validation job to CI workflow#41603
DeepDiver1975 merged 2 commits into
masterfrom
fix/calens

Conversation

@DeepDiver1975

@DeepDiver1975 DeepDiver1975 commented Jun 8, 2026

Copy link
Copy Markdown
Contributor

Summary

Add a Changelog lint CI job that runs calens on every PR and master push. This catches malformed changelog entries in changelog/unreleased/ before they can block the auto-changelog generation workflow on master (as happened in https://github.com/owncloud/core/actions/runs/27135471818).

Changes

  • New calens job in .github/workflows/ci.yml
  • Uses actionhippie/calens@0b8ceba55a72f5ef8b7d910698fcd28be8bb8c19 (v1.13.4) — SHA-pinned to match the pin already used in the reusable calens.yml workflow
  • Runs on both PRs and master pushes via the existing on: trigger block

Test plan

  • The "Changelog lint" check appears and passes on this PR

🤖 Generated with Claude Code

DeepDiver1975 and others added 2 commits June 8, 2026 15:33
Run calens on every PR and master push to catch malformed changelog
entries before they block the changelog generation workflow.

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Signed-off-by: Thomas Müller <1005065+DeepDiver1975@users.noreply.github.com>
Comment thread changelog/unreleased/41597

@DeepDiver1975 DeepDiver1975 left a comment

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

Code Review — ci: add calens validation job to CI workflow

Overview: Adds a Changelog lint CI job using calens to validate changelog/unreleased/ entries on every PR and master push. Also fixes a malformed changelog entry in changelog/unreleased/41597 (Fix:Bugfix:).

CI job

calens:
  name: Changelog lint
  runs-on: ubuntu-latest
  steps:
    - uses: actions/checkout@df4cb1c069e1874edd31b4311f1884172cec0e10 # v6.0.3
    - uses: actionhippie/calens@0b8ceba55a72f5ef8b7d910698fcd28be8bb8c19 # v1.13.4

Both actions are SHA-pinned ✅ — consistent with the existing calens.yml reusable workflow policy. The job runs under the existing on: triggers (PR + master push). ✅

OSPO Actions policy: The new agents.md file (added in #41567) states workflows may only use actions owned by owncloud, created by GitHub (actions/*), or verified in the GitHub Marketplace. actions/checkout is GitHub-owned ✅. actionhippie/calens — this is the same action already used in the repo's reusable calens.yml workflow, so it's an established pattern rather than a new third-party addition. ✅

Changelog fix

Fix:Bugfix: in changelog/unreleased/41597 — this is exactly the kind of malformed entry the new job catches. Having it fixed in this same PR demonstrates the value of the linter. ✅

Summary

Aspect Assessment
SHA-pinned actions ✅ Both pinned
Consistent with existing calens usage ✅ Same action/SHA as reusable workflow
Changelog fix bundled ✅ Demonstrates the linter's value

Verdict: Ready to merge.

@DeepDiver1975
DeepDiver1975 requested a review from phil-davis June 8, 2026 14:45
@DeepDiver1975
DeepDiver1975 merged commit 8e3a274 into master Jun 8, 2026
13 checks passed
@DeepDiver1975
DeepDiver1975 deleted the fix/calens branch June 8, 2026 14:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants