Skip to content

Periodic orcid citation update failing due to approver requirements #71

Open
@vincerubinetti

Description

@vincerubinetti

https://github.com/greenelab/greenelab.com/actions/runs/3819357107

remote: error: At least 1 approving review is required by reviewers with write access.

It makes sense that you'd want an approver for pull requests, but unfortunately that means the GitHub Action that periodically runs the automatic citation process to update e.g. ORCID papers will fail because it doesn't have permissions.

Apparently there is no way to make an exception for GitHub Actions (yet, but it is a heavily requested feature):
community/community#13836

I could maybe make it such that the action opens a PR instead of committing directly to main, but then I'd imagine them really piling up over time. Maybe I could try to check if a PR for that is already open and update/add to it instead of creating a new one, but that gets a lot more complex.

@cgreene Thoughts? I'm not just thinking about greenelab.com, but the template in general.

I'm thinking we just bet on GitHub implementing the Actions exception within the next year (since it's so heavily requested), and until then, say that people can't have both branch protection rules and scheduled action runs.

Activity

cgreene

cgreene commented on Jan 6, 2023

@cgreene
Member

I agree with the current bet:

I'm thinking we just bet on GitHub implementing the Actions exception within the next year (since it's so heavily requested), and until then, say that people can't have both branch protection rules and scheduled action runs.

If the deploy happened off github pages and only the main was protected, that'd avoid this issue (which, likely, is why manubot manuscripts don't get hit by it). However, that's a lot of additional work and setup would be more painful.

cgreene

cgreene commented on Jan 6, 2023

@cgreene
Member

We can unprotect this repo - no concerns on that. I'm also ok with opening a PR

cgreene

cgreene commented on Jan 7, 2023

@cgreene
Member

I unprotected the branch, but I'm also coming around to the idea of opening a pull request so it becomes obvious what is being added.

vincerubinetti

vincerubinetti commented on Jan 7, 2023

@vincerubinetti
ContributorAuthor

Maybe we can try it in greenelab.com as a pilot and see how it goes. I would probably want to reduce the frequency from weekly to monthly if we're opening up PRs. Given how I've seen other people use/maintain their website, I think most people wouldn't keep up with weekly.

Maybe you could do a twitter poll?

Question regarding Lab Website Template. Which of these more closely matches how would you want your latest papers automatically pulled in from ORCID (or other meta-sources):

A) Weekly, and committed directly to the main branch of your website
B) Monthly, in a new opened pull request

Hopefully this action will make it easy, so I could include both methods in the workflow and just comment one out by default (whichever is less popular).

cgreene

cgreene commented on Jan 11, 2023

@cgreene
Member

I have used that action - happy to help out if you want it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Metadata

Metadata

Assignees

No one assigned

    Labels

    No labels
    No labels

    Type

    No type

    Projects

    No projects

    Milestone

    No milestone

    Relationships

    None yet

    Development

    No branches or pull requests

    Issue actions

      Periodic orcid citation update failing due to approver requirements · Issue #71 · greenelab/greenelab.com