<@U9VHUPVPX> Hi, ive noticed I'm unable to merge t...
# pact-brokerc
C
11/27/2022, 5:11 PM@Beth (pactflow.io/Pact Broker/pact-ruby) Hi, ive noticed I'm unable to merge things without having approvals. not sure if this is because of an overarching org policy. i dont think im able to change it on the repo level either
https://github.com/pact-foundation/pact-broker-chart/pull/16
y
Yousaf Nabi (pactflow.io)
11/29/2022, 1:05 PM
Hey @C are you sorted on this dude? If not drop me a message and we can get you perms. I think the deal was you were happy to be admin / maintain the repo. We've had a historic policy of keeping the original authors as admins of their project on transfer to the pact-foundation org, so it may just be some mixed perms.
We still warrant a full audit and review of the policies at some point but that is some yak shaving for another day
🙌 1
c
C
11/29/2022, 6:58 PMHi @Yousaf Nabi (pactflow.io) as far as I can see, I still think I've got problems merging without approvals on the repo
y
Yousaf Nabi (pactflow.io)
11/29/2022, 7:09 PM
np, will get you sorted tomorrow, just clocking off the eve now. thanks again buddy
🙌 1
c
C
11/29/2022, 9:16 PMCheers @Yousaf Nabi (pactflow.io) đź‘Ť
b
Beth (pactflow.io/Pact Broker/pact-ruby)
11/29/2022, 10:39 PM
I’ll have a look. I didn’t add any rules, but maybe there are some by default.
Beth (pactflow.io/Pact Broker/pact-ruby)
11/29/2022, 10:43 PM
Ok, I’ve added “helm chart maintainers” as able to override the PR approval process.
Beth (pactflow.io/Pact Broker/pact-ruby)
11/29/2022, 10:43 PM
Give it a go when you have time @C
c
C
11/30/2022, 11:45 AM@Beth (pactflow.io/Pact Broker/pact-ruby) All golden 🙂 Thanks again
2 Views