widelands-dev team mailing list archive
-
widelands-dev team
-
Mailing list archive
-
Message #09589
Re: [Merge] lp:~7010622-q/widelands/topple-sounds-6 into lp:widelands
Regarding sound Documentation:
1. Please use underscores instead of spaces for filenames
2. I am unsure about the place where you put it. The doc-folder is mainly for automatic documentation of code (e.g. description of Lua scripting API). I think the documentation is better stored in data/sound. Or we could make it a part of https://wl.widelands.org/docs/wl/ Then it has to be modified to fit with the sphinx syntax.
3. The documentation is somehow a description of your personal contribution to widelands. Maybe it should contain all sound files?
But it is good that you put in the license information, the name of author and the format :-)
--
https://code.launchpad.net/~7010622-q/widelands/topple-sounds-6/+merge/316814
Your team Widelands Developers is requested to review the proposed merge of lp:~7010622-q/widelands/topple-sounds-6 into lp:widelands.
References