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

Please add Andrew G. Dick as contributor #1039

Open
DonaldMcIntyre opened this issue Dec 20, 2022 · 21 comments
Open

Please add Andrew G. Dick as contributor #1039

DonaldMcIntyre opened this issue Dec 20, 2022 · 21 comments

Comments

@DonaldMcIntyre
Copy link
Contributor

This issue is to request to add Andrew G. Dick, marketing manager at the ETC Cooperative, so he may review and approve new article PRs to provide some admin redundancy to make sure we can publish our posts on time.

His GitHub profile is:

https://github.com/andrewgdick

@ghost
Copy link

ghost commented Dec 20, 2022

If we are going to allow more ETC coop reviewers, shouldn't we increase the number of reviews as well?
It's true that from the independent reviewers there's not always the swiftest response and that needs to be improved.
ETC Cooperative reviews the articles internally so ideally we would like to peer review before publishing on the website.

@TheCrowbill
Copy link
Member

While I have no objections to adding another reviewer from ETC Cooperative, nor with Mr. Dick being being that addition, I agree with @W1go that it would provide an opportunity for a single organization coup.

To counter this, I support adding an additional independent reviewer from the community at large at the same time and increasing the number of reviews required for merges by at least one for the website and both Twitter-together repos.

@DonaldMcIntyre
Copy link
Contributor Author

That is fair and reasonable. I also think that if ETC Coop writes and reviews and approves its own posts then it loses the checks and balances.

This was just an idea for the Christmas and New Year's season as several independent admins may be traveling and visiting family.

So, I will close this PR and keep insisting over social media and here to admins to review the posts so we don't fall behind with the weekly schedule (1 content unit each Tuesday, Wednesday, and Thursday).

Thank you!

@TheCrowbill
Copy link
Member

@DonaldMcIntyre please reopen this issue. It will give the other maintainers a chance to catch up or comment.

Also, not all of us can add new members to the organization. I think that is set for only the organization's owner and admins.

@DonaldMcIntyre
Copy link
Contributor Author

@DonaldMcIntyre please reopen this issue. It will give the other maintainers a chance to catch up or comment.

Also, not all of us can add new members to the organization. I think that is set for only the organization's owner and admins.

Reopened.

@TheCrowbill
Copy link
Member

TheCrowbill commented Dec 20, 2022 via email

@bobsummerwill
Copy link
Member

What are the current required reviewer counts for website and Twitter?

@bobsummerwill
Copy link
Member

More non-Coop reviewers would be great, if we can find more volunteers. I would be reticent to raise the required counts too high, though, because it introduces real friction.

@bobsummerwill
Copy link
Member

bobsummerwill commented Dec 21, 2022

In terms of fears about “capture” it is worth remembering that this review process has no potential for “taking over” any administrative functions or doing anything very bad. Reviewers are not super-users. The worst they can do it publish “bad stuff”, but there is a clear paper-trail to demonstrate that abuse and it is easily rectified after the fact.

@bobsummerwill
Copy link
Member

bobsummerwill commented Dec 21, 2022

We absolutely do not want a situation where the Coop is just “pushing to prod” blindly with no quality gate from the rest of the community. This is NOT the Coop website. We are just seeking to be good citizens by contributing into this pool, rather than “hoarding content”, wherever that makes sense. But that absolutely must be done in a consensual and collaborative way.

Just to clarify - is there a specific bounded group of accounts which are able to progress a review? Or can any GitHub account do so?

@bobsummerwill
Copy link
Member

bobsummerwill commented Dec 21, 2022

If we cannot get these articles reviewed in time with acceptable “neutrality” then maybe we just have some days without content from Coop? We have had weeks or months without new content for the Coop in the past, so it would not be a disaster.

Step away from the keyboards, ETC, and spend some time with your families! 😀

CC @andrewgdick

@TheCrowbill
Copy link
Member

What are the current required reviewer counts for website and Twitter?

For the website, two.
For eth_classic Twitter, two.
For etc_network Twitter, one.

The only requirements on reviewers are membership in the organization and write access. A reviewer may be specifically requested by a PR's author, but if no requests, any of us with write access suffice.

@bobsummerwill
Copy link
Member

Right, thanks.

So there is an approval process here where the admins of the ethereumclassic org would need to invite specific accounts to join the org? But we have a potentially unbounded amount of those, and they can do little damage.

@TheCrowbill
Copy link
Member

Yes there is a process but it is low impact. Organization guidelines stipulate that in order to be given the necessary access, one must:

  • have materially contributed to the ETC protocol or community. (How is not mandated.)
  • have 2FA setup on your GitHub account.
  • make oneself known as an anti-Sybill measure. (Again, the method for this is not specific.)

As to the potential for damage, I don't think anyone is suggesting the Cooperative is planning to harm the protocol or the community. That said, this community remembers the past as we watched trusted stewards become bad actors. I don't care to make our missteps over again.

There is no downside to having more editorial eyes on a project prior to publishing. So long as the minimum number of required reviews remains reasonable, raising them by one or two would be more beneficial than burdensome.

@bobsummerwill
Copy link
Member

"That said, this community remembers the past as we watched trusted stewards become bad actors. I don't care to make our missteps over again."

100%.

We need to be doing everything we can be as systemically resilient to internal and external social attacks as we can.

To my mind, our largest remaining weak spots all lie where we use centralized platforms which have all-powerful "admin accounts" where multiple individuals all have super-powers, including over each other. Examples being GitHub (ethereumproject org) and Twitter (eth_classic). I see no good solutions there unless the platforms themselves change to provide multi-sig style admin support. Discord has the same weakness.

Short of that, we only have redundancy on those platforms as a (weak) mitigation.

@ghost
Copy link

ghost commented Dec 21, 2022

In terms of fears about “capture” it is worth remembering that this review process has no potential for “taking over” any administrative functions or doing anything very bad. Reviewers are not super-users. The worst they can do it publish “bad stuff”, but there is a clear paper-trail to demonstrate that abuse and it is easily rectified after the fact.

This is not about fear of a capture, it's mainly about how peer review can improve the content of the already high value content that @DonaldMcIntyre is producing. Also very often mistakes or unclarities seep in because of the elevated knowledge of the specialist reviewers. Sometimes theories are taken for fact and should be modified before publication. Once published the content gets spread to different platforms and the stories start to live their own lives. Undoing is a waste of energy and I'm a proponent of peer review.

Look this is only my reflection, everyone can tune it, I'm just a voice in this ecosystem.

@bobsummerwill
Copy link
Member

I would agree on getting the content right prior to publication, @w1g0. Review is very important.

I think that @DonaldMcIntyre's "let get this out and then we'll fix little things later" (which I saw somewhere and cannot find now - maybe in review for one of the pending PRs) was actually more related to a bug/issue in the Netlify workflow where simple changes get stuck in the preview build and we are not sure how they get unstuck.

Looks like this:
https://app.netlify.com/sites/jocular-bienenstitch-b37dca/deploys/63a23ec7efdcc60009fd884b

@IstoraMandiri
Copy link
Collaborator

IstoraMandiri commented Dec 22, 2022

As @TheCrowbill mentioned, to keep that process of onboarding new approvers neutral, there are certain criteria matched as discussed above and at https://github.com/ethereumclassic/volunteer#on-boarding. As long as these requirements are met, anyone should be able to apply in that repo and we can discuss individual merits in that repo.

Specific to this website repo, I'm against requiring an additional review for merging content to master, regardless of whether andrew is added to the approvals team.

Coop, having both bob and donald with review perimssions, already have the ability to approve content if they wanted to by submitting a PR from an external account and having them both approve. With andrew also being added, this would require 4 approvals to avoid, not 3, which is definitely too many IMO.

As mentioned by @bobsummerwill , the damage of malicious pushing is minimal and undoable with a clear paper trail, so I think there's not much risk here. The negative is long term annoyance of having 3 maintainers approve every little change or bugfix, which can take days already (with just two approvals).

@bobsummerwill
Copy link
Member

Thanks, @IstoraMandiri.

With regard to adding Andrew, I am deliberately trying to remove myself from the loop on as much stuff as possible anyway, so it would functionally be more of a matter of Andrew replacing me. The whole point of having a comms team is for me to not be needed.

It is certainly preferable to have non-Coop people reviewing wherever possible.

I would agree on that negative to an additional reviewer. Things are slow as it is.

@IstoraMandiri
Copy link
Collaborator

@DonaldMcIntyre @w1g0 @TheCrowbill @bobsummerwill

I created a new issue in the volunteers repo above to track adding andrew to the github org.

@ghost
Copy link

ghost commented Dec 23, 2022

I would agree on that negative to an additional reviewer. Things are slow as it is.

Convenience is the Achilles' heel of security.

@IstoraMandiri IstoraMandiri added this to Content in Istora TODOs Apr 16, 2023
@IstoraMandiri IstoraMandiri removed this from Content in Istora TODOs Apr 16, 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

4 participants