Skip to content

chore(ci): validate Renovate configuration - #1587

Merged
bradclawsie merged 3 commits into
mainfrom
chore/renovate-config-validation
Oct 5, 2026
Merged

bradclawsie merged 3 commits into
mainfrom
chore/renovate-config-validation

Conversation

@slaskawi

@slaskawi slaskawi commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

Description

Run the shared Renovate validator from hk checks in CI.

Related Issue

Not applicable.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Other (security config, docs update, etc)

Checklist before merging

Signed-off-by: Sebastian Łaskawiec <sebastian.laskawiec@defenseunicorns.com>
Copilot AI balanced review requested due to automatic review settings October 5, 2026 06:11
@slaskawi
slaskawi requested a review from a team as a code owner October 5, 2026 06:11

Copilot AI left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

Copilot review overview

🟢 Approval recommended

The validator is correctly wired into CI with the required pinned Node.js tooling available.

Review effort: Balanced
Findings: None

What changed in this PR

Adds Renovate configuration validation to the repository’s standard CI lint task.

Changes:

  • Updates the shared lint tasks to uds-common v1.30.1.
  • Adds and invokes the Renovate validator during linting.
File Description
tasks/​lint.yaml Exposes the shared Renovate validation task.
tasks.yaml Runs Renovate validation in the main lint task.

💡 Add a code-review agent skill or configure MCP servers for context-aware, tailored reviews. Learn more in the docs.

@greptile-apps

greptile-apps Bot commented Oct 5, 2026 •

Copy link
Copy Markdown
Contributor

RetriggerConfidence Score: 4/5

[Medium risk] Adds a Renovate configuration validation step to the build.

The PR should not merge until the new validator checks the repository’s actual Renovate configuration.

Findings

  1. P1 Validator targets the wrong file ▶

Summary

The PR adds shared Renovate validation to the repository lint task and updates its uds-common task include to v1.30.1.

  • The new invocation needs to target this repository’s root-level Renovate configuration.

Reviews (1) · Last reviewed commit: "chore(ci): validate Renovate configurati..."

Comment thread tasks/lint.yaml Resolved
Signed-off-by: Sebastian Łaskawiec <sebastian.laskawiec@defenseunicorns.com>
Comment thread tasks.yaml Resolved
Signed-off-by: Sebastian Łaskawiec <sebastian.laskawiec@defenseunicorns.com>
@bradclawsie
bradclawsie merged commit ded5010 into main Oct 5, 2026
54 of 55 checks passed
@bradclawsie
bradclawsie deleted the chore/renovate-config-validation branch October 5, 2026 12:04
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