pyOpenSci / pyosMeta

A package that updates pyOpenSci contributor and package metadata on our website
BSD 3-Clause "New" or "Revised" License
4 stars 17 forks source link

Add: CI test that runs all open reviews (maybe weekly) to identify issues before we accept a package #184

Open lwasser opened 1 month ago

lwasser commented 1 month ago

Right now we have a streamlined workflow where we update contributors in the pyopensci.github.io repo using a cron job that runs

This allows us to keep all of our current contributor data and review data current on the website.

It would also be useful to be able to run a command that gets all currently open reviews i have code here that does this BUT because we just updated our API, that code likely needs to be modified.

It could be that the update here is

  1. modify update-reviews to take a input of type single or array that represents labels. then we could do something like

    • update-reviews -label pyos-accepted to get all reviewers with that label name pyos-accepted
    • update-reviews -type closed to get all closed reviewers
    • update-reviews -labels array-of-labels-here -type closed
  2. then we could run an additional step in CI that would get all open reviews, filter out issues that are not reviews

I am not sure what approach here is most efficient but we do need to test open reviews so we know if there are bugs in our API due to how people fill out the issue forms before we accept a package! i am very open to what this workflow looks like.