numixproject / numix-core

Builder for App Icon Themes
GNU General Public License v3.0
766 stars 146 forks source link

Update Maintainers List #5722

Closed Foggalong closed 3 years ago

Foggalong commented 3 years ago

Updates the Code Owners list so that @palob also has ownership rights and (should) be able to merge PRs without my approval. Also added an explicit notice about who the "maintainers" are for this repository, will add similar to others.

Foggalong commented 3 years ago

Tangential, but I've given the @numixproject/icons team maintainer rights to these repositories - think that should give everyone more rights to transfer issues between repos, use moderation tools, etc.

Screenshot from 2021-04-14 05-24-10

palob commented 3 years ago

So this is the new situation:

Foggalong commented 3 years ago

Huh that is weird, I'm not sure why you can't overrule my shit or push straight to master because I thought we were on equal footing now permissions wise. I'll do some digging in the settings and work out what the mismatch is there, thanks for flagging!

palob commented 3 years ago

Direct committing to the main branch works! Will check how are things with PRs during the next week.

Foggalong commented 3 years ago

Awesome, thanks so much @palob! :heart:

palob commented 3 years ago

Can merge PRs from Numix branches, from fork branches and I see the merge button in branches with your pending disapproving reviews :+1:

Foggalong commented 3 years ago

Eeeexcellent :heart: