Skip to content

[releases/28.x] Update CODEOWNERS to remove permissions entries#7841

Open
mazhelez wants to merge 1 commit into
releases/28.xfrom
backport/releases/28.x/7830/20260423131610
Open

[releases/28.x] Update CODEOWNERS to remove permissions entries#7841
mazhelez wants to merge 1 commit into
releases/28.xfrom
backport/releases/28.x/7830/20260423131610

Conversation

@mazhelez
Copy link
Copy Markdown
Contributor

@mazhelez mazhelez commented Apr 23, 2026 •
edited by github-actions Bot
Loading

This pull request backports #7830 to releases/28.x

Fixes AB#632234

Copilot AI review requested due to automatic review settings April 23, 2026 11:17
@mazhelez mazhelez requested a review from a team as a code owner April 23, 2026 11:17
@github-actions github-actions Bot added this to the Version 28.1 milestone Apr 23, 2026
Copilot AI reviewed Apr 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

Copilot AI left a comment

Choose a reason for hiding this comment

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

Pull request overview

Backports PR #7830 to releases/28.x by updating the repository’s CODEOWNERS configuration to stop routing ownership for certain permissions-related paths/filetypes to the permissions owners group.

Changes:

  • Removed CODEOWNERS entries for /src/**/Permissions/ and /src/**/permissions/.
  • Removed CODEOWNERS entries for *.Permissionset.al and *.Permissionsetext.al.

💡 Add Copilot custom instructions for smarter, more guided reviews. Learn how to get started.

@mazhelez mazhelez enabled auto-merge (squash) April 23, 2026 11:34
elbardel
elbardel approved these changes Apr 23, 2026
View reviewed changes
Copy link
Copy Markdown
Contributor

@elbardel elbardel left a comment

Choose a reason for hiding this comment

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

LGTM - same CODEOWNERS cleanup across branches.

@mazhelez mazhelez mentioned this pull request Apr 23, 2026
Open
@mazhelez mazhelez closed this May 9, 2026
auto-merge was automatically disabled May 9, 2026 14:31

Pull request was closed

@mazhelez mazhelez reopened this May 9, 2026
@github-actions
Copy link
Copy Markdown

github-actions Bot commented May 15, 2026

⚠️ Stale Status Check Deleted

The Pull Request Build workflow run for this PR was older than 72 hours and has been deleted.

📋 Why was it deleted?

Status checks that are too old may no longer reflect the current state of the target branch. To ensure this PR is validated against the latest code and passes up-to-date checks, a fresh build is required.

🔄 How to trigger a new status check:

  1. 📤 Push a new commit to the PR branch, or
  2. 🔁 Close and reopen the PR

This will automatically trigger a new Pull Request Build workflow run.

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

Reviewers

Copilot code review Copilot Copilot left review comments

@darjoo darjoo darjoo approved these changes

@JesperSchulz JesperSchulz JesperSchulz approved these changes

@elbardel elbardel elbardel approved these changes

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

Version 28.1

Development

Successfully merging this pull request may close these issues.

5 participants

@mazhelez @darjoo @JesperSchulz @elbardel