widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #01989
Re: [Merge] lp:~hjd/widelands/disabled-s390-patch into lp:~widelands-dev/widelands/debian
Review: Approve
I looked through this and I wondered why you did not also delete the patch file. Unused files in a repository are usually a bad idea, since they are forgotten over time. If somebody needs it back in the future, they can reverse-cherrypick your commit.
Otherwise lgtm.
--
https://code.launchpad.net/~hjd/widelands/disabled-s390-patch/+merge/217293
Your team Widelands Developers is subscribed to branch lp:~widelands-dev/widelands/debian.
References