-
Notifications
You must be signed in to change notification settings - Fork 14
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
jhudsl-robot is needed for syncs #774
base: main
Are you sure you want to change the base?
Conversation
No broken url errors! 🎉 |
No spelling errors! 🎉 |
Re-rendered previews from the latest commit:
* note not all html features will be properly displayed in the "quick preview" but it will give you a rough idea. Updated at 2024-09-23 with changes from the latest commit f73bede |
OCS might already have the robot set up though- I vaguely remember doing that - wasn't sure if you wanted to test a way to do syncs without the robot or not though |
The robot already has access: https://github.com/orgs/opencasestudies/outside-collaborators |
@@ -23,8 +23,8 @@ jobs: | |||
- name: Login as github actions bot | |||
run: | | |||
git config --global --add safe.directory $GITHUB_WORKSPACE | |||
git config --global user.name 'github-actions[bot]' | |||
git config --global user.email 'github-actions[bot]@users.noreply.github.com' | |||
git config --local user.email "[email protected]" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
It's good jhudsl-robot has access to OCS but because I changed this part it wasn't actually getting access @carriewright11
Summary
AH. So there is one place that jhudsl-robot is needed. It's for syncs. See #773
So #728 didn't consider this part.
We'll need to add it back to the documentation and issue template too.
Feedback
I haven't tested this on anything. We need something that is external and that jhudsl-robot is given permissions to. I'll probably attempt to send to OCS or something.