When a post involving reviewing code is encountered, any of these two things can happen:
- one or more users leave a comment, telling the OP to post on CR
- one or more users flag the post for migration
As I've observed, the former causes the most problems. Sometimes the user is not quite right in making the recommendation (which is understandable if it's a gray area area), and sometimes the user is dead wrong. This is why I stress the importance of getting familiar with CR's Help Center, especially before commenting. The OP will almost always take that advice and post on CR right away, even without reading the Help Center. We then have to tell them that their post is off-topic there, which could leave them disappointed. Of course, we'll have to correct the commenter(s) as well.
Flagging, on the other hand, is safer because the SO mods are the "gateway" to migration. The other CR mods and I regularly stay in touch with the SO mods, looking over questions that have been flagged for migration. This helps educate the SO mods more, as well as the flaggers.
At this time, it doesn't matter which you choose, as long as you're sure that the post should be migrated. If you're unsure, then you don't have to do anything. Bad migrations just create more work for us, and it's not something that should be our primary focus.
code-review
,review
andreviews
. Not sure if the latter two are for code reviews at all, since they have no description, but I've seenreview
used that way at least once. Thecode-review
tag does mention that CR is for "otherwise working" code, but it's not really emphasised, and is silent on what should be done with reviews of, e.g. architecture or class structures which haven't yet been fleshed out into full code.