If you scroll up to my first comment, you'll see a green "Show diff comments" link. Click on it and you'll see my comments in the diff. For longer diffs, I use the browser's search function and search for the reviewer's username to skip through the comments.
I agree with your reasoning about an "is writable" check, let's keep this branch as is. I'd like my diff comments addressed though.
The automatic builds are triggered by every push to a branch that has an active merge request.
For our testing environment, including our codecheck, see:
https:/ /wl.widelands. org/wiki/ RegressionTests /
If you scroll up to my first comment, you'll see a green "Show diff comments" link. Click on it and you'll see my comments in the diff. For longer diffs, I use the browser's search function and search for the reviewer's username to skip through the comments.
I agree with your reasoning about an "is writable" check, let's keep this branch as is. I'd like my diff comments addressed though.