I recently earned the privilege to review First Posts, so I tried it a little. I passed a test once and I got a case where I failed. I think it is very good to have a system to test people like me who are beginner in reviewing.
In my opinion, this post is not bad, but can be improved:
- The title "Constructor inheritance and direct member initialisation" doesn't reflect the question "So my question is now, is this valid C++ 11?". The title should be more precise and mention the compile error.
- The error log is contained in a code format, but this post advises to use quote format.
- The users edited his question, but he added "Edit" and "2nd edit:". In my opinion it's unnecessary to add that when editing a question. This information is already present in edit log of the question.
This post say that tests are auto-generated. I think it's a mistake. A test should be written by a human, especially a test to test a human.
Like I say, I don't say that this post is bad. It's a good post for a new user. So I wanted to leave a comment asking him to improve his post. Why is it considered to be a failed test when I leave a comment?