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
Member

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
Member 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

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants