Skip to content
This repository has been archived by the owner on Sep 19, 2024. It is now read-only.

Create CODEOWNERS and tw-rules to eventually archive repo #21

Merged
merged 2 commits into from
Sep 18, 2024
Merged

Conversation

louis-pie
Copy link
Contributor

@louis-pie louis-pie commented Sep 16, 2024

Context

Create CODEOWNERS and tw-rules to eventually archive repo AP-1837

amofakhar
amofakhar previously approved these changes Sep 16, 2024
@wise-github-bot-app
Copy link

The approval(s) from amofakhar do(es)n't fullfill the approvers requirements because:

  • The approver's cost centre, ENGPLA, maps to the PLATFORM business function. As the code that was changed is owned by ENGINEERING, this approval won't satisfy our separation of duties check. We'll need an additional approval from someone in ENGINEERING. This approval may still help satisfy other codeowner requirements.

@louis-pie louis-pie changed the title Create CODEOWNERS to comply and eventually archive Create CODEOWNERS and tw-rules to eventually archive repo Sep 16, 2024
@wise-github-bot-app
Copy link

The approval(s) from mhassanzahid do(es)n't fullfill the approvers requirements because:

  • The approver's cost centre, ENGPLA, maps to the PLATFORM business function. As the code that was changed is owned by ENGINEERING, this approval won't satisfy our separation of duties check. We'll need an additional approval from someone in ENGINEERING. This approval may still help satisfy other codeowner requirements.

@wise-github-bot-app
Copy link

The approval(s) from MatthewStocks do(es)n't fullfill the approvers requirements because:

  • The approver's cost centre, ENGSEC, maps to the GOVERNANCE business function. As the code that was changed is owned by ENGINEERING, this approval won't satisfy our separation of duties check. We'll need an additional approval from someone in ENGINEERING. This approval may still help satisfy other codeowner requirements.

@MatthewStocks
Copy link

/wise-bot emergency-merge Preparing repository for archiving

@wise-github-bot-app wise-github-bot-app bot added the change:emergency Change made as part of a declared incident label Sep 18, 2024
@wise-github-bot-app
Copy link

🟢

Emergency merge requested by MatthewStocks.

This event has been added to the security log. The merge could take few seconds...

Remember that, once it is merged, a maintainer should review the code and approve it retrospectively by commenting /wise-bot approve within 14 days to avoid an SLA breach.

Requester MatthewStocks
Reason Preparing repository for archiving
Merge Method squash

@tw-census tw-census merged commit 93a533d into main Sep 18, 2024
11 of 12 checks passed
@tw-census tw-census deleted the AP-1837 branch September 18, 2024 16:02
@wise-github-bot-app
Copy link

Your tw-rules.yaml file has changed. Syncing your branch protection settings:

Branch pattern Applied Comment
master 🟡 Warning. The branch does not exist, but protection has been applied to given pattern.
main 🟢 Branch succesfully protected.

You have more info in the docs.

@wise-github-bot-app
Copy link

Your CODEOWNERS or the tw-rules.yaml file has changed. Syncing your GitHub teams with your CODEOWNERS file and extraWriters in tw-rules:

Collaborator Name Status Comment
analytics-platform 🟢 Team updated with role owner
machine-users 🟢 Team updated with role push
service-platform 🟡 Unexpected Team would be deleted with strict access
koszti 🟡 Unexpected User would be deleted with strict access
full-time-technical-staff 🟢 Team was already up to date

You have more info in the docs

@mhassanzahid
Copy link

/wise-bot approve

@wise-github-bot-app
Copy link

Approved by mhassanzahid

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
change:approved change:emergency Change made as part of a declared incident
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants