Fix UML permissions from forks / Run only on merge to main #967
Conversation
…ng into fix-uml-permissions
|
Couldn't figure out the permissions as it would require the fork's owner to have the action bot permissions. Because of that, I just have it that if only runs when pymc-labs is the owner. In a fork's case, it should only run when it is merged in. Which still keeps it up to date. How does that sounds @juanitorduz ? An alternative is to only run on merge to main. Do you have a preference? |
|
Personally, running only on main merge might be fine so that developer doesn't have to |
The change on this PR looks good! Still, running only on main merge seems more convenient as you suggest, Feel free to push the change. |
wd60622
changed the title
…#967) * change repo and python file change to kick off * comment to check for failure * using the repo * delete to trigger * the previous trigger * push up again * check for the owner being pymc-labs * remove unused * only run after merge to main
* change repo and python file change to kick off * comment to check for failure * using the repo * delete to trigger * the previous trigger * push up again * check for the owner being pymc-labs * remove unused * only run after merge to main
* change repo and python file change to kick off * comment to check for failure * using the repo * delete to trigger * the previous trigger * push up again * check for the owner being pymc-labs * remove unused * only run after merge to main
Description
Related Issue
Checklist
Modules affected
Type of change
📚 Documentation preview 📚: https://pymc-marketing--967.org.readthedocs.build/en/967/