Skip to content

chore(release): promote v1.52.0 - #652

Merged
fredcamaral merged 7 commits into
mainfrom
develop
Aug 6, 2026
Merged

chore(release): promote v1.52.0#652
fredcamaral merged 7 commits into
mainfrom
develop

Conversation

@fredcamaral

Copy link
Copy Markdown
Member
Lerian

GitHub Actions Shared Workflows


Description

Promotes the validated develop state to main for the stable v1.52.0 release.

This release adds backward-compatible Breaking Change Guard result support to the PR validation reporter and Actions summary composites. Direct consumers that omit the result preserve their current rendering and mergeability behavior.

The mandatory pr-validation integration will always supply this result and will not expose a guard opt-out. That enforcement lands only after this prerequisite is available through stable v1.

Develop published v1.52.0-beta.1 successfully: https://github.com/LerianStudio/github-actions-shared-workflows/actions/runs/31081933688

Type of Change

  • feat: New workflow or new input/output/step in an existing workflow
  • fix: Bug fix in a workflow (incorrect behavior, broken step, wrong condition)
  • perf: Performance improvement (e.g. caching, parallelism, reduced steps)
  • refactor: Internal restructuring with no behavior change
  • docs: Documentation only (README, docs/, inline comments)
  • ci: Changes to self-CI (workflows under .github/workflows/ that run on this repo)
  • chore: Dependency bumps, config updates, maintenance
  • test: Adding or updating tests
  • BREAKING CHANGE: Callers must update their configuration after this PR

Breaking Changes

None.

Testing

  • YAML syntax validated locally
  • Triggered a real workflow run on a caller repository using @this-branch or the beta tag
  • Verified all existing inputs still work with default values
  • Confirmed no secrets or tokens are printed in logs
  • Checked that unrelated workflows are not affected

Caller repo / workflow run: Develop beta release v1.52.0-beta.1: https://github.com/LerianStudio/github-actions-shared-workflows/actions/runs/31081933688

Related Issues

Related PR: #651.

lerian-studio and others added 6 commits August 6, 2026 06:45
- expose has-breaking-change-guard output on reporter and summary composites
- render only exact skipped as skip icon; cancelled maps to failure icon
- document breaking-change-result derivation in reporter usage example

Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@fredcamaral
fredcamaral requested a review from a team as a code owner August 6, 2026 07:46
@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 4119157b-925e-449a-a394-605f57052ec1

📥 Commits

Reviewing files that changed from the base of the PR and between 6df08fc and 4ae1d87.

📒 Files selected for processing (1)
  • src/notify/pr-validation-reporter/action.yml

Summary by CodeRabbit

  • New Features
    • Added optional Breaking Change Guard results to pull request validation reports and summaries.
    • Summaries display the guard status when available.
    • Added an output indicating whether the guard was evaluated.
  • Bug Fixes
    • Cancelled checks are now treated as failures, while unknown statuses receive a warning indicator.
    • Non-success strict checks now correctly block mergeability.

Walkthrough

The validation summary and PR reporter now accept breaking-change-result. They expose guard presence, render the guard conditionally, and treat non-success strict results as blocking. Documentation and composite-action examples describe the new wiring.

Changes

Breaking-change result reporting

Layer / File(s) Summary
Validation summary integration
src/validate/pr-checks-summary/action.yml, src/validate/pr-checks-summary/README.md
The action accepts the breaking-change result, exposes whether the guard ran, maps additional statuses, and conditionally adds the guard row to the summary.
PR reporter mergeability integration
src/notify/pr-validation-reporter/action.yml, src/notify/pr-validation-reporter/README.md
The reporter accepts and forwards the breaking-change result, adds the guard to strict checks when present, and blocks every non-success strict result.

Estimated code review effort: 3 (Moderate) | ~20 minutes

Possibly related PRs

Suggested reviewers: bedatty

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly identifies the stable v1.52.0 release promotion and matches the primary change.
Description check ✅ Passed The description follows the template and documents the release purpose, behavior changes, compatibility, testing, and validation run.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check.
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests
  • Commit unit tests in branch develop

Comment @coderabbitai help to get the list of available commands.

@lerian-studio lerian-studio added size/S PR changes 50–199 lines documentation Improvements or additions to documentation notify Changes to notification composite actions (src/notify/) validate Changes to PR validation composite actions (src/validate/) composite Changes to any composite action manifest (src/**/*.yml) labels Aug 6, 2026
@lerian-studio

lerian-studio commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🔍 Lint Analysis

Check Files Scanned Status
YAML Lint 2 file(s) ✅ success
Action Lint no changes ⏭️ skipped
Pinned Actions 2 file(s) ✅ success
Markdown Link Check 2 file(s) ✅ success
Spelling Check 4 file(s) ✅ success
Shell Check 2 file(s) ✅ success
README Check 2 file(s) ✅ success
Composite Schema 2 file(s) ✅ success
Deployment Matrix no changes ⏭️ skipped

🔍 View full scan logs

@lerian-studio

lerian-studio commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🔍 PR Validation Summary

✅ PR Mergeable — no blocking failures

Check Status Blocking
Source Branch ✅ success yes
PR Title ✅ success yes
PR Description ✅ success yes
PR Size ✅ success no
Auto Labels ✅ success no
PR Metadata ✅ success no

🔍 View workflow run

@lerian-studio

lerian-studio commented Aug 6, 2026

Copy link
Copy Markdown
Contributor

🛡️ CodeQL Analysis Results

Languages analyzed: actions

✅ No security issues found.


🔍 View full scan logs | 🛡️ Security tab

@coderabbitai coderabbitai Bot left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Warning

CodeRabbit couldn't request changes on this pull request because it doesn't have sufficient GitHub permissions.

Please grant CodeRabbit Pull requests: Read and write permission and re-run the review.

👉 Steps to fix this

Actionable comments posted: 2

🤖 Prompt for all review comments with AI agents
Verify each finding against current code. Fix only still-valid issues, skip the
rest with a brief reason, keep changes minimal, and validate.

Inline comments:
In `@src/notify/pr-validation-reporter/action.yml`:
- Around line 32-35: Update the breaking-change-result input description in the
action metadata to include cancelled alongside success, failure, and skipped,
matching the reporter behavior and existing README contract. Leave the default
and other input settings unchanged.

In `@src/validate/pr-checks-summary/README.md`:
- Around line 23-24: Add the breaking-change guard producer, resolver, job
output, and normalized result wiring to .github/workflows/pr-validation.yml,
then pass that output to both consumers:
src/validate/pr-checks-summary/README.md:23-24 requires the summary action
input, and src/notify/pr-validation-reporter/README.md:24 requires the reporter
action input. Preserve the existing skipped fallback only for omitted or
optional direct consumers; the live pr-validation workflow must always provide
the guard result.
🪄 Autofix

Fix all unresolved CodeRabbit comments on this PR:

  • Push a commit to this branch (recommended)
  • Create a new PR with the fixes

ℹ️ Review info
⚙️ Run configuration

Configuration used: Path: .coderabbit.yml

Review profile: ASSERTIVE

Plan: Pro

Run ID: 59985e2f-5ab8-4a1b-9d7e-f980e068df14

📥 Commits

Reviewing files that changed from the base of the PR and between 62414c5 and 6df08fc.

📒 Files selected for processing (4)
  • src/notify/pr-validation-reporter/README.md
  • src/notify/pr-validation-reporter/action.yml
  • src/validate/pr-checks-summary/README.md
  • src/validate/pr-checks-summary/action.yml

Comment thread src/notify/pr-validation-reporter/action.yml
Comment thread src/validate/pr-checks-summary/README.md
Co-authored-by: Codesmith <codesmith-bot@users.noreply.github.com>
@fredcamaral

Copy link
Copy Markdown
Member Author

@CodeRabbit -- approve

@coderabbitai

coderabbitai Bot commented Aug 6, 2026

Copy link
Copy Markdown

@fredcamaral Approved.

✅ Action performed

Comments resolved and changes approved.

@fredcamaral
fredcamaral merged commit 4da6537 into main Aug 6, 2026
33 checks passed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

composite Changes to any composite action manifest (src/**/*.yml) documentation Improvements or additions to documentation notify Changes to notification composite actions (src/notify/) size/S PR changes 50–199 lines validate Changes to PR validation composite actions (src/validate/)

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants