ci: update pullapprove

[ocombe]

PR Type

What kind of change does this PR introduce?

[x] CI related changes

What is the current behavior?

Tobias is still listed on pullapprove but he's not working on Angular anymore

What is the new behavior?

Removed Tobias from the list and promoted secondaries to primary when possible, or @chuckjaz when relevant

Does this PR introduce a breaking change?

[x] No

Other information

We need to talk about promoting more people for pullapprove because some packages don't even have a secondary anymore

[gkalpak]

I think @alxhub is primary now.

[chuckjaz]

Corect alxhub is now primary for benchpress.

[ocombe]

ok nice I'll update it

[vicb]

On a side note I think that pullapprove functionality is now builtin into GH. Could you take a quick look at pros/cons @ocombe ?

[ocombe]

From what I understand based on their very short doc it is pretty similar. The main difference is that github will automatically ask a review from the code owners when the PR touches their code. Pullapprove lets you add whoever you want.
This is a big difference for us, because we add multiple owners per branch as backups (usually Igor & Misko) just in case the primary/secondary owners are not there.
We don't want to ask all of the owners for each PR, only the primary or secondary is fine. That would create a lot of noise for nothing.

[angular-automatic-lock-bot]

This issue has been automatically locked due to inactivity.
Please file a new issue if you are encountering a similar or related problem.

Read more about our automatic conversation locking policy.

_{This action has been performed automatically by a bot.}