10

You have just asked a question on Meta that: has a title that is clear, specific, and concise has an introduction that provides great context, and examples of what your expectations are based on it provides information on why you did what you did, and what you expected the results to be. In short, it's a reasonably well formulated, and presented question. ...


8

The banner you are seeing contains an explanation. You also got a comment from one of the users involved in closing your question that explicitly asks you to "elaborate what the instructions are to accomplish". To phrase that in a little less complicated English: We can see that you have some code. It seems to be doing what you want it to do. It's ...


8

If there were a mechanism (?) to impose an edit freeze while discussion happens, I'd vote for that, but I don't know that there is one. Actually... there is a mechanism for that. It's just not available to regular users. For cases where edits keep getting applied back and forth, diamond moderators have the ability to lock a post. Before this happens it can ...


8

Currently your question has a score of +0/-3, and there is one close vote against your question for our Authorship of Code close reason: Authorship of code: Since Code Review is a community where programmers improve their skills through peer review, we require that the code be posted by an author or maintainer of the code, that the code be embedded directly,...


7

Like @1201ProgramAlarm mentioned in the comment The problem is that the code to be reviewed must be embeded directly in the question. As it stands, there is no code in the question and nothing to review. , which is clearly stated in the close-reason with an additional link to a meta-answer providing additional context about why this is needed. For a ...


7

On-topic, discouraged Posting solutions to the same programming challenge in multiple languages in one question is discouraged. Posting multiple solutions is not off-topic, whether it be in one or more posts. As such the discouragement is unlikely to be accompanied by a slap on the wrist - closure, downvotes, comments. Note: It is encouraged to leave some ...


7

I don't know the policy well enough to say whether it explicitly prohibits this, but my opinion is: If the fundamental question is solid enough, then I wouldn't recommend closure - we already have a large-enough closure rate, and I wouldn't advocate yet one more reason to push well-meaning users away. Instead, I would offer the advice to the OP that there ...


7

For a question to be on-topic it must have a compiler, interpreter or travel through a series of transpilers until we get something we can run. For example JSX + TypeScript -> JavaScript is fine. Whilst the OP would more than happily post pseudocode, we won't allow it. Since it's not pseudocode we can happily forget about this rule, as thinking about it ...


6

First of all, you're allowed to post the whole thing as-is, all in the same question. 13,000 characters is well within the limit of what a post may contain (65,536 characters as a hard limit), leaving plenty of room for a project description. A description should ideally contain: what problem does your project solve what prompted you to write what are your ...


6

Your title doesn't match the site's standard. Is this an appropriate use of async/await combined with Task continuation? The title is what you want out of the review however pretty much all Code Review questions would be titled the same as yours and is generally not great. As such you should change the title to what your code does. There is additional ...


5

Users that are new to the network may not intuitively understand that they do not own any of the content they post on Stack Exchange, as per terms of service all user content is licensed under CC-BY-SA, and as content hosted on the SE network it is the duty of network users to make sure all network content looks great and reads well, and this is incentivized ...


4

I think you've missed a lot of context from your question, which can be found in chat. Lack of response to your question Should I close this question? Based on the lack of response, I am concluding it is off topic. Is that accurate? Nintendo Nightmare [any%] Speedrun Bot (17:37) – https://chat.stackexchange.com/transcript/message/57551595#57551595 If your ...


4

Your question isn't entirely clear, but I'll try to answer it since I suspect what you're really looking for is a combination of the following. Pseudocode is off-topic. There's multiple reasons for this, but one of them is that the code has to be actual code, from a project or anything, that's actually used or has been used. In this case, the code presented ...


4

While there currently appears to be nothing in the rules against this practice, I would recommend making separate posts. Some reviewers may only be familiar one language though others may be familiar with multiple. The biggest advantage I see to separate posts is that there is a potential for more reputation and badges, which opens the door to more ...


4

Code that's not included can not be reviewed. That's simple. "I don't want to separate it into independent questions because the components altogether form the whole architecture; I think that the entire context is important here." Considering you've posted the general overview and context in your first question, that question can serve as extra ...


3

I do not have diamond mod privileges on this site, so I cannot see the full history of deleted comments, but if I recall correctly... I found your question by searching for php questions (not the review queue). I think I voted to close because the reviewable content needs to be embedded in the question. I think I posted a comment to explicitly state that ...


3

Thank you for calling us out on this. From my perspective One would think that after 3352 CV reviews a user in the review queues should know what he or she is doing. I admit that I was the fifth reviewer and I came into it, seeing that it had: 1 CV for missing context 2 votes for migration I.I.R.C. 1 vote for needs more detail. Regretfully I didn't fully ...


2

I think there are multiple levels of issues / reviews. Character level review. These are the 'petty' points such as style guide conformance and whitespace issues. You should have spaces on both sides of operators. Line level review. These are normally pretty basic changes and focus on how to improve the readability of a specific line. It's more Pythonic ...


1

... someone just closed it. Closing a question on Code Review takes 5 votes from members of the community that have more than 3000 rep points. This makes the closure a community decision. Only a moderator can close a question with one vote and that is usually rare. Once the question is edited it is entered into a reopen vote queue and will need at least 3 ...


1

Is it okay to update code of an off-topic question using code from an answer? Yes The answerer has provided the code under the current user contributions license, so no concern for the user being a copyright troll. And what harm is there to Code Review by fixing the problem for the user? Can the code from the answer be used to update the code in the ...


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