chore: fix Renovate preset reference#100
Conversation
|
@coderabbitai autofix |
Behavior ChangedRenovate configuration will now resolve its base preset from the correct repository. The Risk Surface
Validation Evidence
Cross-Repository ImplicationsThis repository now explicitly depends on the external preset at WalkthroughUpdated ChangesRenovate Configuration Base Update
Estimated code review effort🎯 1 (Trivial) | ⏱️ ~2 minutes Possibly related PRs
Suggested labels
🚥 Pre-merge checks | ✅ 18 | ❌ 2❌ Failed checks (2 warnings)
✅ Passed checks (18 passed)
✏️ Tip: You can configure your own custom pre-merge checks in the settings. ✨ Finishing Touches🧪 Generate unit tests (beta)
✨ Simplify code
Warning Review ran into problems🔥 ProblemsLinked repositories: Your configuration references 17 linked repositories, but your current plan allows 10. Analyzed Comment |
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. Autofix skipped. No unresolved CodeRabbit review comments with fix instructions found. |
Up to standards ✅🟢 Issues
|
![codacy-production[bot]](https://avatars.githubusercontent.com/in/56611?s=60&v=4){kind=small}
There was a problem hiding this comment.
Pull Request Overview
This PR successfully updates the Renovate configuration in renovate.json to point to the correct repository path for its preset, replacing the invalid reference. The code is up to standards according to Codacy analysis, and the implementation fully satisfies the intended requirement.
Test suggestions
- Validate that renovate.json extends from the correct GitHub repository
TIP Improve review quality by adding custom instructions
TIP How was this review? Give us feedback
![coderabbitai[bot]](https://avatars.githubusercontent.com/in/347564?s=60&v=4){kind=small}
There was a problem hiding this comment.
Actionable comments posted: 1
🤖 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 `@renovate.json`:
- Line 3: The renovate.json extends property was changed to
"github>ANcpLua/renovate-config" but other repos and docs still reference the
old preset; update the extend string in the other repositories
(ANcpLua.Analyzers, ANcpLua.NET.Sdk, ANcpLua.Roslyn.Utilities) to match the new
preset ("github>ANcpLua/renovate-config") and also update any documentation and
workflow comments (e.g., AGENTS.md and workflow files) that reference
"github>O-ANcppLua/renovate-config" so all references are consistent and won’t
404; coordinate the commits/PRs across those repos so the change rolls out
simultaneously.
🪄 Autofix (Beta)
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.yaml
Review profile: ASSERTIVE
Plan: Pro Plus
Run ID: 0ba70d7f-5821-4fca-bc76-9383c048a8b4
📒 Files selected for processing (1)
renovate.json
📜 Review details
⏰ Context from checks skipped due to timeout of 900000ms. You can increase the timeout in your CodeRabbit configuration to a maximum of 15 minutes (900000ms). (4)
- GitHub Check: build (ubuntu-latest)
- GitHub Check: build (windows-latest)
- GitHub Check: build (macos-latest)
- GitHub Check: Codacy Static Code Analysis
🧰 Additional context used
📓 Path-based instructions (1)
**/*.{cs,ts,tsx,js,jsx,json,yaml,yml,props,csproj}
📄 CodeRabbit inference engine (.github/copilot-instructions.md)
Skip files whose first ~3 lines contain
// Ported from <upstream>,// Generated, or// Auto-generated— surface a one-line note instead of line-level findings
Files:
renovate.json
🔀 Multi-repo context ANcpLua/ANcpLua.Analyzers, ANcpLua/ANcpLua.NET.Sdk, O-ANcppLua/ANcpLua.OtelConventions.Api, ANcpLua/ANcpLua.Roslyn.Utilities, ANcpLua/Arqio, ANcpLua/ErrorOrX, O-ANcppLua/Nuke.OpenTelemetry.Conventions, ANcpLua/Paperless
ANcpLua/ANcpLua.Analyzers
- renovate.json — extends "github>O-ANcppLua/renovate-config" [::ANcpLua/ANcpLua.Analyzers::renovate.json:3]
- AGENTS.md — documentation link referencing O-ANcppLua/renovate-config [::ANcpLua/ANcpLua.Analyzers::AGENTS.md:10]
- .github/workflows/auto-merge.yml — comment referencing the O-ANcppLua preset [::ANcpLua/ANcpLua.Analyzers::.github/workflows/auto-merge.yml:6]
ANcpLua/ANcpLua.NET.Sdk
- renovate.json — extends "github>O-ANcppLua/renovate-config" [::ANcpLua/ANcpLua.NET.Sdk::renovate.json:3]
- AGENTS.md — documentation link referencing O-ANcppLua/renovate-config [::ANcpLua/ANcpLua.NET.Sdk::AGENTS.md:9]
- .github/workflows/auto-merge.yml — comment referencing the O-ANcppLua preset [::ANcpLua/ANcpLua.NET.Sdk::.github/workflows/auto-merge.yml:6]
O-ANcppLua/ANcpLua.OtelConventions.Api
- renovate.json — already extends "github>ANcpLua/renovate-config" (no change needed) [::O-ANcppLua/ANcpLua.OtelConventions.Api::renovate.json:3]
ANcpLua/ANcpLua.Roslyn.Utilities
- renovate.json — extends "github>O-ANcppLua/renovate-config" [::ANcpLua/ANcpLua.Roslyn.Utilities::renovate.json:3]
- AGENTS.md — documentation link referencing O-ANcppLua/renovate-config [::ANcpLua/ANcpLua.Roslyn.Utilities::AGENTS.md:9]
- .github/workflows/auto-merge.yml — comment referencing the O-ANcppLua preset [::ANcpLua/ANcpLua.Roslyn.Utilities::.github/workflows/auto-merge.yml:6]
ANcpLua/Arqio
- src/Arqio.DependencyTools/renovate.json — uses "local>ANcpLua/renovate-config" (local preset reference) [::ANcpLua/Arqio::src/Arqio.DependencyTools/renovate.json:4]
- CHANGELOG.md — mentions ANcpLua/renovate-config PRs in history (contextual) [::ANcpLua/Arqio::CHANGELOG.md:10,24]
ANcpLua/ErrorOrX
- renovate.json — uses "local>ANcpLua/renovate-config" [::ANcpLua/ErrorOrX::renovate.json:4]
O-ANcppLua/Nuke.OpenTelemetry.Conventions
- renovate.json — extends "github>ANcpLua/renovate-config" (already pointing to ANcpLua) [::O-ANcppLua/Nuke.OpenTelemetry.Conventions::renovate.json:3]
ANcpLua/Paperless
- renovate.json — uses "local>ANcpLua/renovate-config" [::ANcpLua/Paperless::renovate.json:4]
Summary / impact
- Multiple ANcpLua/* repos (ANalyzers, NET.Sdk, Roslyn.Utilities) currently reference the old "github>O-ANcppLua/renovate-config" and will benefit from the same repoint-to-ANcpLua change to avoid 404s. Some repos already point to ANcpLua/renovate-config or use local>ANcpLua/renovate-config and do not need this fix.
🔇 Additional comments (1)
renovate.json (1)
3-3: ⚡ Quick winPreset reference is valid and comprehensive.
The new preset
github>ANcpLua/renovate-configexists, is publicly accessible, and contains well-designed configuration explicitly for ANcpLua repos (including ANcpLua.Agents). The preset includes 37 packageRules covering NuGet stability policies, Microsoft Agent Framework maturity tracking (stable/RC/preview/alpha quarantine), first-party package automerge, and extensive Version.props custom managers. Fixes the old broken 404 reference.
| { | ||
| "$schema": "https://docs.renovatebot.com/renovate-schema.json", | ||
| "extends": ["github>O-ANcppLua/renovate-config"], | ||
| "extends": ["github>ANcpLua/renovate-config"], |
There was a problem hiding this comment.
🧹 Nitpick | 🔵 Trivial
Cross-repository consistency: other repos still reference the old preset.
Linked repositories ANcpLua.Analyzers, ANcpLua.NET.Sdk, and ANcpLua.Roslyn.Utilities still extend github>O-ANcppLua/renovate-config and will continue to 404 until updated. Documentation files (AGENTS.md) and workflow comments also reference the old preset. Consider coordinating updates across the ecosystem to prevent configuration drift.
🤖 Prompt for 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.
In `@renovate.json` at line 3, The renovate.json extends property was changed to
"github>ANcpLua/renovate-config" but other repos and docs still reference the
old preset; update the extend string in the other repositories
(ANcpLua.Analyzers, ANcpLua.NET.Sdk, ANcpLua.Roslyn.Utilities) to match the new
preset ("github>ANcpLua/renovate-config") and also update any documentation and
workflow comments (e.g., AGENTS.md and workflow files) that reference
"github>O-ANcppLua/renovate-config" so all references are consistent and won’t
404; coordinate the commits/PRs across those repos so the change rolls out
simultaneously.
|
@coderabbitai autofix |
|
Claude encountered an error after 2s —— View job I'll analyze this and get back to you. |
|
Note Autofix is a beta feature. Expect some limitations and changes as we gather feedback and continue to improve it. ⛔ CodeRabbit does not have permission to push changes to this repository. Please ensure CodeRabbit has Contents: Read and write permission. |
ANcpLua
dismissed
coderabbitai[bot]’s stale review
Resolved: the cross-repo cleanup CodeRabbit flagged has been completed in parallel — Analyzers#162, Roslyn.Utilities#128, claude-plugins#250 all merged; 7 other repos direct-pushed via API. This PR completes the set.
1 participant
Switch
extendsfromO-ANcppLua/renovate-config(which 404s) to canonicalANcpLua/renovate-config. Auto-merge will land this once required checks pass.