Skip to content

Conversation

@Bashamega
Copy link
Contributor

No description provided.

@github-actions
Copy link
Contributor

github-actions bot commented Nov 7, 2025

Thanks for the PR!

This section of the codebase is owned by @saschanaz - if they write a comment saying "LGTM" then it will be merged.

@saschanaz
Copy link
Contributor

LGTM, thanks for the cleanup

@github-actions
Copy link
Contributor

There was an issue merging, maybe try again saschanaz. Details

@jakebailey
Copy link
Member

I thought the merging was fixed but apparently not; will check the settings when I'm back in again 😞

@saschanaz
Copy link
Contributor

LGTM

@github-actions github-actions bot merged commit 8338cdf into microsoft:main Nov 23, 2025
7 checks passed
@github-actions
Copy link
Contributor

Merging because @saschanaz is a code-owner of all the changes - thanks!

@saschanaz saschanaz deleted the remove-entries branch November 23, 2025 21:35
@saschanaz
Copy link
Contributor

Error: HttpError: New changes require approval from someone other than the last pusher.

Maybe that's it? But I'm not sure that includes me as the approver.

@jakebailey
Copy link
Member

I disabled that check so I'm not sure what the deal is

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.

3 participants