Code review comment for lp:~widelands-dev/widelands/bug-1753230-working-with-tempfiles

Revision history for this message
GunChleoc (gunchleoc) wrote :

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.

« Back to merge proposal