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

Use "last active 12 months ago" for trust measurement #5

Open
decentral1se opened this issue Jan 2, 2023 · 1 comment
Open

Use "last active 12 months ago" for trust measurement #5

decentral1se opened this issue Jan 2, 2023 · 1 comment

Comments

@decentral1se
Copy link
Member

decentral1se commented Jan 2, 2023

E.g. https://github.com/ssbc/.github/actions/runs/3819652633/jobs/6497388690#step:2:1251 mycognosist should be removed, we don't do it. Things keep failing. New folks hack on go-ssb a lot and should get brought into the org. But the CI is failing so they don't get suggested? Should we switch to a warning for this kind of stuff? It always involves a human looking at it and making a decision, so, seems fair? /cc @mycognosist @staltz

@staltz
Copy link
Member

staltz commented Jan 2, 2023

The CI job has run to completion, so it hasn't missed the fact that someone (e.g. new go-ssb contributors) should be added to the org.

The reason why the algo is suggesting mycognosist to be removed is that his TrustNet Org score has dropped a bit over time, dipping a bit under the threshold. I have postponed to remove mycognosist for two reasons: (1) he has been active lately in go-ssb issues so clearly helping, and could volunteer to merge PRs etc, (2) the TrustNet Org algo seems like it needs to be updated because I think the "trust" measurement shouldn't decrease over time, but currently it's decreasing for a lot of people, including me and arj (who are the top 2 at the moment), and that doesn't make sense; I think it makes more sense to use "last active 12 months ago" as the threshold to remove a person. This is something we should discuss when we update the rules of membership this year.

@decentral1se decentral1se changed the title afaict ignoring failures blocks new people getting added? Use last active 12 months ago for trust measurement Jan 2, 2023
@decentral1se decentral1se changed the title Use last active 12 months ago for trust measurement Use "last active 12 months ago" for trust measurement Jan 2, 2023
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

No branches or pull requests

2 participants