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 action to audit publicly listed organizations #1941

Merged
merged 5 commits into from
Nov 15, 2023
Merged

Conversation

IanButterworth
Copy link
Member

@IanButterworth IanButterworth commented Sep 29, 2023

It seems to be reasonable to expect any publicly listed julia organization to have at least 2 public members.

This sets up an action to check that whenever the list is changed in community/organizations.md, or if the action is manually triggered, otherwise it won't run to avoid noise on unrelated PRs.

Screenshot 2023-09-29 at 2 25 16 PM

TODO:

@github-actions
Copy link

Once the build has completed, you can preview your PR at this URL: https://julialang.netlify.app/previews/PR1941/ in ~15 minutes

@IanButterworth IanButterworth force-pushed the ib/audit_orgs branch 4 times, most recently from 0a93710 to 47b5b65 Compare September 29, 2023 17:54
@IanButterworth IanButterworth force-pushed the ib/audit_orgs branch 7 times, most recently from 09b7054 to 0041b9f Compare September 29, 2023 18:23
@StefanKarpinski
Copy link
Member

Very good idea!

@IanButterworth
Copy link
Member Author

IanButterworth commented Oct 11, 2023

@StefanKarpinski do you think community/organizations.md should also have some sort of notice like:

Organizations listed here should strive to have at least 2 owners with admin rights, and at least 2 public members. Doing so helps to protect the organization's repositories against a single person being unreachable.

@DilumAluthge
Copy link
Member

DilumAluthge commented Oct 11, 2023

at least 2 members with admin rights

It might be clearer to change "members with admin rights" to "organization owners".

@ViralBShah
Copy link
Member

Looks great. Let's get it merged.

@IanButterworth
Copy link
Member Author

Ok. What do you think about #1941 (comment)

@ViralBShah
Copy link
Member

Ok. What do you think about #1941 (comment)

That feels totally reasonable. Let's do it.

@IanButterworth IanButterworth marked this pull request as ready for review November 15, 2023 01:18
@IanButterworth
Copy link
Member Author

I'll go ahead with this. At this point, this is the status

Finding orgs with fewer than 2 public memers that are listed in https://julialang.org/community/organizations/
 - JuliaPraxis 1 members
 - JuliaActors 0 members
 - JuliaFirstOrder 1 members
 - JuliaNLSolvers 1 members
 - JuliaNonconvex 1 members
 - JuliaActuary 1 members
 - ModiaSim 1 members
 - JuliaAstrodynamics 1 members
 - juliahci 1 members
 - JuliaQuantum 1 members

@IanButterworth IanButterworth merged commit 23901d4 into main Nov 15, 2023
1 of 2 checks passed
@IanButterworth IanButterworth deleted the ib/audit_orgs branch November 15, 2023 01:29
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

5 participants