Skip to content

MPT-26022 remove CODEOWNERS - #436

Merged
svazquezco merged 1 commit into
mainfrom
feature/MPT-26022/remove-codeowners
Oct 14, 2026
Merged

svazquezco merged 1 commit into
mainfrom
feature/MPT-26022/remove-codeowners

Conversation

@svazquezco

@svazquezco svazquezco commented Oct 2, 2026 •

Copy link
Copy Markdown
Collaborator

🤖 AI-generated PR — Please review carefully.

What

  • Remove .github/CODEOWNERS so GitHub no longer requests @softwareone-platform/sirius as reviewer automatically on every PR. PR activity is now announced in Microsoft Teams.
  • Branch protection on main does not require code owner reviews, so merging is not affected.

Jira: MPT-26022

Closes MPT-26022

  • Remove .github/CODEOWNERS to stop automatic review requests to @softwareone-platform/sirius.

@svazquezco
svazquezco requested a review from a team as a code owner October 2, 2026 11:16
@svazquezco
svazquezco requested review from alephsur and ruben-sebrango and removed request for a team October 2, 2026 11:16
@github-actions

github-actions Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

✅ Found Jira issue key in the title: MPT-26022

Generated by 🚫 dangerJS against 936bf2d

@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

Important

Review skipped

Review was skipped as selected files did not have any reviewable changes.

⚙️ Run configuration
  • Configuration used: Repository YAML (base), Organization UI (inherited)
  • Review profile: CHILL
  • Plan: Essentials
  • Run ID: b414f29f-02a3-4719-b777-d29faac84dbb
📥 Commits

Reviewing files that changed from the base of the PR and between b3a4ef6 and 936bf2d.

You can disable this status message by setting the reviews.review_status to false in the CodeRabbit configuration file.

Use the checkbox below for a quick retry:

  • 🔍 Trigger review

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository YAML (base), Organization UI (inherited)

Review profile: CHILL

Plan: Essentials

Run ID: b7416fd5-87ee-45d1-8d39-cc15d340bf37

📥 Commits

Reviewing files that changed from the base of the PR and between 6c01d66 and e426d88.

📒 Files selected for processing (1)
  • .github/CODEOWNERS
🔗 Linked repositories identified

CodeRabbit considers these linked repositories for cross-repo context during reviews:

💤 Files with no reviewable changes (1)
  • .github/CODEOWNERS

Included review availability: This review used your included allowance. 4 included reviews remain after this review. Your included PR review attempts over the past 7 days set your current allowance at 5 reviews per hour.

📜 Recent review details
⏰ Context from checks skipped due to timeout. (1)
  • GitHub Check: build
🧰 Additional context used
🔀 Multi-repo context softwareone-platform/mpt-extension-skills

Linked repositories findings

softwareone-platform/mpt-extension-skills

  • The linked repository retains its own .github/CODEOWNERS assigning @softwareone-platform/sirius; this change does not alter that repository’s ownership configuration. [::softwareone-platform/mpt-extension-skills::]
  • Shared pull-request and branching standards remain owned by the Sirius Team (standards/pull-requests.md, standards/branching.md). [::softwareone-platform/mpt-extension-skills::]
  • Teams notification guidance supports on-demand PR-ready announcements only after all checks pass and CodeRabbit approves; it does not describe automatic notification for every PR activity (skills/mpt-ext-task-notify-pr-ready-in-teams/SKILL.md:3,10-16, skills/mpt-ext-workflow-notify-pr-ready/SKILL.md:3,10-16). [::softwareone-platform/mpt-extension-skills::]
  • No references to MPT-26022 or a cross-repository consumer of the removed CODEOWNERS entry were found. [::softwareone-platform/mpt-extension-skills::]
📝 Walkthrough

The .github/CODEOWNERS file no longer assigns code ownership to @softwareone-platform/sirius.

Changes: CODEOWNERS Update

Layer / File(s) Summary
Remove code-owner entry
.github/CODEOWNERS
The file no longer assigns code ownership to @softwareone-platform/sirius.

Priority: ⬇️ Low

Estimated code review effort: 1 (Trivial) | ~3 minutes

Merge Risk: ⚪ Minimal · up to e426d

No actionable issue is identified with removing the automatic code-owner review request; the change is mergeable after normal checks.

🚥 Pre-merge checks | ✅ 2 | ❌ 1

❌ Failed checks (1 warning)

Check name Status Explanation Resolution
Documentation Up To Date ⚠️ Warning The PR changes repository developer workflow by removing .github/CODEOWNERS, which stops automatic @softwareone-platform/sirius review requests. The review range contains only that deletion and no… Update docs/contributing.md to document the current pull request review workflow: code-owner review requests are no longer automatic, and pull request activity is announced in Microsoft Teams. Keep the documentation change in the same PR.
✅ Passed checks (2 passed)
Check name Status Explanation
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.
Full details: Documentation Up To Date

Explanation

The PR changes repository developer workflow by removing .github/CODEOWNERS, which stops automatic @softwareone-platform/sirius review requests. The review range contains only that deletion and no documentation update. docs/contributing.md is the mapped document for developer workflow changes and states that repository behavior changes require the narrowest relevant documentation update.

  • Autopilot · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

Autopilot is currently an internal CodeRabbit preview.

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

@svazquezco
svazquezco force-pushed the feature/MPT-26022/remove-codeowners branch from e426d88 to b3a4ef6 Compare October 5, 2026 09:21
Remove .github/CODEOWNERS so GitHub no longer requests the team review
automatically on every pull request. Pull request activity is now
announced in Microsoft Teams, and branch protection does not require
code owner reviews.
@svazquezco
svazquezco force-pushed the feature/MPT-26022/remove-codeowners branch from b3a4ef6 to 936bf2d Compare October 14, 2026 12:53
@svazquezco
svazquezco merged commit 2f7280f into main Oct 14, 2026
3 checks passed
@svazquezco
svazquezco deleted the feature/MPT-26022/remove-codeowners branch October 14, 2026 12:55
@sonarqubecloud

Copy link
Copy Markdown

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