Skip to content

ci: route Swashbuckle (Swagger) Renovate updates to Platform#7978

Merged
amorask-bitwarden merged 2 commits into
mainfrom
chore/route-swashbuckle-renovate-to-platform
Jul 14, 2026
Merged

ci: route Swashbuckle (Swagger) Renovate updates to Platform#7978
amorask-bitwarden merged 2 commits into
mainfrom
chore/route-swashbuckle-renovate-to-platform

Conversation

@amorask-bitwarden

Copy link
Copy Markdown
Contributor

🎟️ Tracking

Follows Renovate PR #7774 ([deps] Billing: Update swashbuckle-aspnetcore monorepo to v10.2.3) and team discussion: Swagger
tooling is out of Billing's scope, and Platform will own it. No Jira ticket.

📔 Objective

PR #7774 was tagged [deps] Billing: because Swashbuckle.AspNetCore and
Swashbuckle.AspNetCore.SwaggerGen were listed under the "Billing owned
dependencies" rule in .github/renovate.json5.

This moves both packages to the "Platform owned dependencies" rule so future
updates route to team-platform-dev with a [deps] Platform: prefix. Only
ownership/routing changes — the packages are added solely to the team-ownership
rule (not the dashboard-approval rule), so PR-vs-dashboard update behavior is
unchanged.

Once merged, #7774 can be closed and regenerated from the Dependency Dashboard
so it re-routes to Platform.

Swashbuckle.AspNetCore and Swashbuckle.AspNetCore.SwaggerGen were listed
under the Billing team-ownership rule, so Renovate tagged the
swashbuckle-aspnetcore monorepo update (#7774) as Billing even though
Swagger is out of Billing's scope. Per team discussion, Platform owns
Swagger; move both packages to the Platform ownership rule so future
updates route to team-platform-dev with a [deps] Platform: prefix.

Co-Authored-By: Claude Opus 4.8 (1M context) <noreply@anthropic.com>
@amorask-bitwarden amorask-bitwarden added the ai-review Request a Claude code review label Jul 14, 2026
@github-actions

github-actions Bot commented Jul 14, 2026

Copy link
Copy Markdown
Contributor

🤖 Bitwarden Claude Code Review

Overall Assessment: APPROVE

This PR moves Swashbuckle.AspNetCore and Swashbuckle.AspNetCore.SwaggerGen from the "Billing owned dependencies" rule to the "Platform owned dependencies" rule in .github/renovate.json5, so future Renovate updates route to team-platform-dev with a [deps] Platform: commit prefix. The move is clean — both packages are removed from the Billing list and added to the Platform list with no duplication, and alphabetical ordering after Quartz is preserved. Neither package appears in the dashboard-approval rule (lines 226-240), so PR-vs-dashboard update behavior is unchanged as the description states. This is a routing-only config change — no dependency versions are added, removed, or bumped, so no security, correctness, or breaking-change concerns apply.

@amorask-bitwarden amorask-bitwarden added the t:ci Change Type - Updates to automated workflows label Jul 14, 2026
@amorask-bitwarden amorask-bitwarden marked this pull request as ready for review July 14, 2026 16:11
@amorask-bitwarden amorask-bitwarden merged commit e7bff14 into main Jul 14, 2026
41 checks passed
@amorask-bitwarden amorask-bitwarden deleted the chore/route-swashbuckle-renovate-to-platform branch July 14, 2026 16:30
@codecov

codecov Bot commented Jul 14, 2026

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 62.05%. Comparing base (04c886b) to head (5f6f24a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@             Coverage Diff             @@
##             main    #7978       +/-   ##
===========================================
+ Coverage   14.89%   62.05%   +47.16%     
===========================================
  Files        1382     2277      +895     
  Lines       60129    99452    +39323     
  Branches     4778     8983     +4205     
===========================================
+ Hits         8956    61719    +52763     
+ Misses      51022    35560    -15462     
- Partials      151     2173     +2022     

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

ai-review Request a Claude code review t:ci Change Type - Updates to automated workflows

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants