44 votes

What would Clippy say?

No code in the question Very easy to test for. If there is not a single formatted full line of code in a question: Code Review questions must include the code to be reviewed in the body of the ...
Kaz's user avatar
  • 8,790
44 votes

Must the code be "clean"?

Make the code as clean as you can before submitting it for review. The more work you put into your code, the more you're likely to get out of review. That's because reviewers (especially the ...
Gareth Rees's user avatar
  • 49.6k
43 votes
Accepted

Distributing "welcome votes" to first posts

Upvote for effort I think that it's important to keep in mind what it's like to post to Code Review as a total newcomer. After all, they don't teach you in school how to post a question to Code ...
200_success's user avatar
38 votes
Accepted

What is the motivation to spend time doing a code review?

You're not wrong. Unfortunately, because we are a smaller site (we're 27th/166 of main sites in the network, and 27th/73 of graduated sites) and we only have ~40,000 questions, it's difficult in some ...
Der Kommissar's user avatar
37 votes

Checklist for how to write a good Code Review question

Simon's Guide for posting a good question There are a lot of questions that appear on Code Review, and all questions are fighting for reviewer attention. This is a non-exhaustive list of items that I ...
Simon Forsberg's user avatar
34 votes
Accepted

Should pedantry be discouraged?

The answer that you are referring to has been voted up quite a bit. I'd like to point out that we cannot really predict or influence how the votes go. People tend to upvote stuff that they find ...
200_success's user avatar
34 votes
Accepted

Do people read other reviews for learning?

So as a mod I have access to some statistics on where traffic for our site comes from. A bit of this information can also be accessed by 25k users. Let's have a bit of a look at some of that data. ...
Vogel612's user avatar
  • 25.2k
34 votes

Phrancis: A final review of a good member

Phrancis. Even though I never met you, I feel like this is the first time I'm losing someone so close to me. A good friend. It's just a few days more than six years ago now that you joined Code Review....
Simon Forsberg's user avatar
33 votes
Accepted

Should I leave in documentation in code for review?

Is it a best practice to include things like function documentation and inline comments when posting code for review? If you normally document your code, then include the documentation when posting ...
Simon Forsberg's user avatar
32 votes

Let's pick t-shirt tagline!

Making code slightly less wrong. Just so the ones loving the existing line can vote to keep it. Credits: @Zak
301_Moved_Permanently's user avatar
32 votes
Accepted

Must the code be "clean"?

No it doesn't have to be clean. It can get reviewed, but people will point those problems out. If you know they are bad you should probably clean them, that way we won't badger on about them, and ...
Peilonrayz's user avatar
  • 43k
32 votes

Code Review is getting boring

If you're finding Code Review boring then it's time to take a break! This is part of the natural cycle of volunteer engagement on a Stack Exchange site — it's fun to start with, but after a while you'...
Gareth Rees's user avatar
  • 49.6k
32 votes

It is with disappointment that I hereby resign as a moderator

I'm very sad to see you go and I want to thank you for being a hugely positive influence to this site in your elected tenure, your appointed tenure, before and in between. I've learned incredibly ...
Vogel612's user avatar
  • 25.2k
31 votes

A friendly challenge!

The title says "A friendly challenge", but really, what this is is a duel. For honor and glory. There is no possible good outcome to a duel. PPCG is going to lose. Then you'll be out for revenge a ...
200_success's user avatar
30 votes

Should you refrain from answering questions that are likely to get closed?

Do not answer questions that are likely to get closed. Such questions are either not ready yet, or not suitable for the site. A question that is not ready may be misinterpreted. You simply cannot ...
janos's user avatar
  • 111k
30 votes
Accepted

Editing out the Ruby tag from a Ruby-on-Rails question

Editing out ruby from ruby-on-rails is basically equivalent to editing out c# or vb.net from asp.net questions (containing code in the relevant language). In short… it's a stupid idea. Please reject ...
Vogel612's user avatar
  • 25.2k
30 votes
Accepted

April 2017 Community Challenge

Simulate a Multi-Way Intersection This was actually a project I had to do in college with PLC's. The idea is to simulate a four-way (or other intersection) stop-light, each light should be ...
Der Kommissar's user avatar
30 votes

Does Stack Overflow count as a third party site?

I don't see any reason to treat other Stack Exchange sites in a special way regarding this issue: Reasons #3 and especially #4 are still very important. Posts in the Stack Exchange network can ...
hoffmale's user avatar
  • 6,468
29 votes
Accepted

June 2016 Community Challenge

Chutes (snakes1) & Ladders When I was a child, the board-game "Chutes & Ladders" really grew my hatred for anything random. The board is 10 x 10. The object of the game is to get your piece ...
Raystafarian's user avatar
  • 7,129
29 votes

Should we stay opted-in for Winter Bash 2016?

It's not simply tradition, it's part of the community's culture. We're one of the few SE sites known for actually liking fun™. Let's keep it that way. Let the Winterbash games commence! May the odds ...
RubberDuck's user avatar
  • 30.8k
29 votes

On the huge amount of work needed for a CR answer

IMO the answering scheme on SO is where the problem is. People answer quickies and get their upvotes, and walk away thinking they've made the Internet a better place. Allow me to make a slight ...
Mathieu Guindon's user avatar
29 votes

25 at twenty-five!

Congratultions, Toby. Perhaps now is a good time to dig up some metrics on the site activity and progress. I built these SEDE queries a while ago, and running them again is a good thing. First up, is ...
rolfl's user avatar
  • 97.3k
28 votes

Let's pick t-shirt tagline!

Because all code can be improved. Except Jon Skeet's
Mathieu Guindon's user avatar
28 votes

What would Clippy say?

Machine Learning There's way too many cases to consider here, we should apply Machine Learning to this to make a machine learn what makes a question likely to be a bad question. I have recently ...
Simon Forsberg's user avatar
28 votes
Accepted

These mysterious $A framework questions

This smells extremely fishy. I did some digging, and I discovered that it's a small world. Not only do many of these users answer each other's questions, they also have a similar style of coding. (...
Laurel's user avatar
  • 666
28 votes
Accepted

On the huge amount of work needed for a CR answer

I think it's unfair to Code Review to view reputation/vote as the metric for success of an answer or the real impact of your answer. The vote/reputation is a metric for Stack Exchange, but I feel here ...
Marc-Andre's user avatar
  • 6,701
28 votes

2018 Moderator Election Q&A - Questionnaire

rolfl's Answers Suppose that you closed a question, which angers the original poster, who calls you a power-tripping moderator. How do you respond? The implicit question is, under what ...
rolfl's user avatar
  • 97.3k
28 votes
Accepted

Code that handles n% of test cases but not all

If the code produces wrong answers for some test cases, then the author is aware that the code is not working correctly as intended, and the question is thus prima facie off-topic for Code Review. ...
200_success's user avatar
28 votes

2021 Community Moderator Election Results

Congratulations to you both, I'm sure you'll make a fine addition to the team. I know the both of you to be very active and very experienced. The site is in good hands.
Mast's user avatar
  • 13.3k
27 votes

Community Promotion Ads - 2016

200_success's user avatar

Only top scored, non community-wiki answers of a minimum length are eligible