Skip to content
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

add symphorien to known users #439

Closed
wants to merge 1 commit into from
Closed

Conversation

@symphorien
Copy link

symphorien commented Mar 16, 2020

No description provided.

@Ekleog
Copy link
Member

Ekleog commented Mar 16, 2020

👍

@veprbl
Copy link
Member

veprbl commented Apr 21, 2020

I was just about to submit the same request.

@grahamc
Copy link
Member

grahamc commented Apr 23, 2020

Thanks for the PR, but no need: all users are considered "known users" now, and being in the "known user" list doesn't grant anyone any special access.

@grahamc grahamc closed this Apr 23, 2020
@veprbl
Copy link
Member

veprbl commented Apr 23, 2020

@grahamc So everyone can build on darwin or not? I think I saw a PR without a darwin build that day.

@veprbl
Copy link
Member

veprbl commented Apr 23, 2020

I guess should target trusted_users in config.public.json instead

@cole-h
Copy link
Member

cole-h commented Apr 23, 2020

Yes, being added to trusted_users in config.public.json is necessary for Darwin builds, while (I believe) everybody can ask ofborg to build aarch64-linux and x86_64-linux.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Linked issues

Successfully merging this pull request may close these issues.

None yet

5 participants
You can’t perform that action at this time.