search

carpentries-incubator / cwl-novice-tutorial

Introduction to Workflows with Common Workflow Language
https://carpentries-incubator.github.io/cwl-novice-tutorial/
Other
11 stars 20 forks source link

enable @mr-c permissions to merge https://github.com/carpentries-incubator/cwl-novice-tutorial/pull/44 #48

Closed mr-c closed 2 years ago

mr-c commented 2 years ago

Ping @MCMaurer

MCMaurer commented 2 years ago

Hey @mr-c, just checking on this- do you not have permissions to merge this PR with Maintain access? If not, I can make you an Admin on the repo.

mr-c commented 2 years ago

Hey @mr-c, just checking on this- do you not have permissions to merge this PR with Maintain access? If not, I can make you an Admin on the repo.

Admin access would ve appreciated, thanks

mr-c commented 2 years ago

@MCMaurer Yeah, looks like I need admin access to approve GitHub Action workflow running for first time contributors? https://github.com/carpentries-incubator/cwl-novice-tutorial/pull/49

In this case I fixed it by force-pushing to the contributors branch with a commit from me, but this will not always be possible

MCMaurer commented 2 years ago

@mr-c got it, just gave you Admin access!