Skip to content

MPT-26022 remove CODEOWNERS - #135

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.

Notes

Jira: MPT-26022

Closes MPT-26022

  • Delete .github/CODEOWNERS to stop automatically requesting @softwareone-platform/sirius as a reviewer on pull requests.

@svazquezco
svazquezco requested a review from a team as a code owner October 2, 2026 12:14
@svazquezco
svazquezco requested review from alephsur and jentyk and removed request for a team October 2, 2026 12:14
@coderabbitai

coderabbitai Bot commented Oct 2, 2026 •

Copy link
Copy Markdown

Review in Change Stack →

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: e6ca80af-a633-483f-9d8f-5ea36c9a263c
📥 Commits

Reviewing files that changed from the base of the PR and between 69fe067 and 6323384.

📒 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
🧰 Additional context used
🔀 Multi-repo context softwareone-platform/mpt-extension-skills

Linked repositories findings

softwareone-platform/mpt-extension-skills

  • .github/CODEOWNERS:1 assigns all files to @softwareone-platform/sirius. Removing it would eliminate the repository’s automatic team-review assignment. [::softwareone-platform/mpt-extension-skills::]
  • standards/pull-requests.md:48-58 documents Danger as the automated PR enforcement mechanism and does not require code-owner approval. [::softwareone-platform/mpt-extension-skills::]
  • .github/workflows/danger.yml:1-23 runs Danger for PR checks; no workflow depends on CODEOWNERS. [::softwareone-platform/mpt-extension-skills::]
  • No references to CODEOWNERS, code-owner reviews, or branch-protection requirements were found in the shared standards, knowledge, or workflow files.
📝 Walkthrough

The .github/CODEOWNERS file was deleted. This removes its ownership assignment for the Sirius team.

Changes: CODEOWNERS Update

Layer / File(s) Summary
Remove owner assignment
.github/CODEOWNERS
The CODEOWNERS entry for the Sirius team was removed.

Priority: ⬇️ Low

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

Merge Risk: ⚪ Minimal · up to 63233

This change removes the automatic review request to the Sirius team and has no runtime impact. It is safe to merge, provided #133 is merged first as the PR description notes.

Pre-merge checks: Passed 2 · Failed 1

Failed Failed checks: 1 warning

Check name Status Explanation Resolution
Documentation Up To Date Warning The PR changes the developer review workflow by deleting .github/CODEOWNERS, which removed the automatic @softwareone-platform/sirius reviewer request. The PR changes only that file (`git diff --n… Update docs/contributing.md in this PR. Document the current pull-request review and notification workflow, including that CODEOWNERS no longer requests @softwareone-platform/sirius automatically and that Teams announces pull-request ac…
Passed 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 the developer review workflow by deleting .github/CODEOWNERS, which removed the automatic @softwareone-platform/sirius reviewer request. The PR changes only that file (git diff --name-status shows D .github/CODEOWNERS; the deleted rule was * @softwareone-platform/sirius). No documentation file changed, and docs/contributing.md does not describe this review-workflow change. The documentation standard maps changed developer workflow to docs/contributing.md (softwareone-platform/mpt-extension-skills, commit 1f162898152a6fd72c6c4d32bdc4cc6bff935193, standards/documentation.md, lines 1-15 and 36-40).

Resolution

Update docs/contributing.md in this PR. Document the current pull-request review and notification workflow, including that CODEOWNERS no longer requests @softwareone-platform/sirius automatically and that Teams announces pull-request activity. If this workflow is intentionally undocumented, explain why it has no repository documentation impact.

  • Fix all pre-merge checks with AI
  • Autofix · Keep fixing CodeRabbit findings and required CI, and resolving merge conflicts

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

@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 6323384

Base automatically changed from feature/MPT-25924/teams-pr-notifications to main October 14, 2026 12:51
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 3482de0 to 6323384 Compare October 14, 2026 12:52
@sonarqubecloud

Copy link
Copy Markdown

@svazquezco
svazquezco merged commit 8840562 into main Oct 14, 2026
6 checks passed
@svazquezco
svazquezco deleted the feature/MPT-26022/remove-codeowners branch October 14, 2026 12:57
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