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

Continue on: "Are unreasonable technical code reviews expected?“ #80

Closed
clivezhg opened this issue Dec 6, 2022 · 1 comment
Closed
Labels

Comments

@clivezhg
Copy link

clivezhg commented Dec 6, 2022

The last thread: #79.
@Ipstenu, since you locked this thread eagerly, I believe you are also the last reviewer.
Becasue you said "You are intentionally opening the door for higher risk in your plugin and refusing to consider you could be wrong" in an earlier email (can we call it "insultation", or "PUA"?), I have been thinking the reviewer is someone else.
However, you never found a real least security issue, you even said something like "no one will read your code", so your points were basically driven by machine scanning.
According to my investigation, most of the hosted plugins are violating your code guidelines, more or less. So, if you are really concerning security as you stated, you should make them re-coding, too.
We think that your review manner should be known by more developers, because most developers expect reasonable code reviews, if they know unreasonable code reviews might be imposed, they might hesitate to submit a plugin, so avoid of wasting time of both sides.

(If you do find real security issue according to your points, as you stated "the security issues are real", you can create issues in my project: https://github.com/clivezhg/Unreasonable-Code-Reviews/. Otherwise, your statement is very pale)

@Ipstenu
Copy link
Contributor

Ipstenu commented Dec 12, 2022

@clivezhg I don't know how to make this more clear for you: This is not a matter of guidelines.

Your issue is with the content of your code review. That's all.

I'm closing this again, because this GitHub repo is literally ONLY FOR THE GUIDELINES. Seriously man, you're in the wrong place.

@Ipstenu Ipstenu closed this as not planned Won't fix, can't repro, duplicate, stale Dec 12, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants