widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #15196
Re: [Merge] lp:~widelands-dev/widelands/bug-1753230-working-with-tempfiles into lp:widelands
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.
--
https://code.launchpad.net/~widelands-dev/widelands/bug-1753230-working-with-tempfiles/+merge/357656
Your team Widelands Developers is requested to review the proposed merge of lp:~widelands-dev/widelands/bug-1753230-working-with-tempfiles into lp:widelands.
References