Skip to content

Support new review template format - process both reviewers list and eic #173

@lwasser

Description

@lwasser

from @pllim

"About that error, looks like pyOpenSci/software-submission#152 (automata) has format of "Reviewers: X, Y, Z" but other issues have "Reviewer 1: X" followed by "Reviewer 2: Y""

We have made a few updates to our review template.

  1. instead of having reviewer 1: and reviewer 2 as separate entries - we have a reviewers field. this field accepts an array of github usernames. Right now the workflow is failing on automata as it uses the new template! We will want to add a check to see what reviewer format the field is provided with (reviewers vs reviewer x). Then we can process the names accordingly. This might require an alias on the pydantic end of things.
  2. We also added the EiC field - make sure we support ingesting that field into our pydantic model & then the output yaml file.

when this is addressed we can then close #77 as well.

Metadata

Metadata

Assignees

No one assigned

    Labels

    help wantedExtra attention is needed

    Type

    No type

    Projects

    Status

    No status

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions