Skip to content
Snippets Groups Projects

Update APKBUILDS to use Maintainer and Co-Maintainer instead of CODEOWNERS

Merged Pablo Correa Gomez requested to merge pabloyoyoista/codeowners-co-maintainers into master

We now have a bot that reads those lines from APKBUILD files, instead of using a proprietary gitlab feature. Relevant changes:

  • Change @ollieparanoid for @Newbyte as maintainer for GNOME packages where Oliver was listed.
  • Don't add @craftyguy as Librem5 maintainer, as he dropped it long ago
  • Remove entries referencing archived ports
  • Don't touch jenneron's maintainership. That's a task for another day
  • Don't add "balejk" as maintainer, since I do not know who that person is. Help figuring it out would be welcomed.
  • Don't add "binarycraft" as maintainer. Last contribution was years ago: https://gitlab.com/binarycraft
  • Don't add more maintainers to linux-postmarketos-qcom-msm8953, as it would have ended up with 7, which might be a bit too much
  • Don't add more maintainers to the PinePhone. The current ones seem to actually be the most active
  • Don't add Danct12 to N900. Sicelo has been single-handedly managing it for a long while

I've tried my best to keep authorship and rules, so that anybody that ever tried to add themselves as maintainers in CODEOWNERS still are maintainers of their packages. However, this has been a more manual process than I would have desired, so I could have made some mistake. Sorry for that, and please let me know in that case.

Fixes #3530 (closed)

Merge request reports

Loading
Loading

Activity

Filter activity
  • Approvals
  • Assignees & reviewers
  • Comments (from bots)
  • Comments (from users)
  • Commits & branches
  • Edits
  • Labels
  • Lock status
  • Mentions
  • Merge request status
  • Tracking
  • Wow thanks for all the work to clean this up! CODEOWNERS was kind of a failed experiment that I feel partly responsible for since I'm the one who proposed it, so I'm really glad to see it being replaced with something that works a lot better :grin:

  • Clayton Craft approved this merge request

    approved this merge request

  • Casey approved this merge request

    approved this merge request

  • achill (fossdd) approved this merge request

    approved this merge request

  • Thank you so much for figuring this all out! I'm glad CODEOWNERS can finally be dropped. :)

  • Ferass El Hafidi approved this merge request

    approved this merge request

  • added 1 commit

    • f91f624d - Update APKBUILDS to use Maintainer and Co-Maintainer instead of CODEOWNERS

    Compare with previous version

  • Thank you everybody for the positive comments!

  • Nikita Travkin approved this merge request

    approved this merge request

  • Oliver Smith approved this merge request

    approved this merge request

  • Robert Eckelmann approved this merge request

    approved this merge request

  • Aster Boese approved this merge request

    approved this merge request

  • Barnabás Czémán requested changes

    requested changes

  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Loading
  • Please register or sign in to reply
    Loading