The proposal to merge lp:~widelands-dev/widelands/doc_type_name into
lp:widelands has been updated.
Status: Needs review => Merged
For more details, see:
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
--
Your team Widelands Developers is subscribed to branch
Thanks for finishing this up :)
@bunnybot merge
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/doc_type_name.
___
Mailing list: http
Ups, sorry for this. :(
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/doc_type_name.
___
Mailing list: https://launchpad.net/~widela
This is a test, please ignore.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/doc_type_name.
___
Mailing list: https://launchpad.net/
Done.
I think this is ready to go now?
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/doc_type_name.
___
Mailing list: https://launc
I think that's a good idea and I agree with the phrasing. I won't get to it
before the weekend, would you like to take care of it?
"You cannot instantiate this directly, access it via wl.Game().map.get_field()
instead."
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge
Continuous integration builds have changed state:
Travis build 2499. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/259871581.
Appveyor build 2323. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_doc_type_n
Nice :-)
I did some additional syntax fixes for wl.map.Warehouse.
There is one issue left from my side:
I the Fields section one could read: "You cannot instantiate this class
directly, instead use wl.map.Map.get_field()." See:
https://wl.widelands.org/docs/wl/autogen_wl_map/#field
The link p
Thanks for digging that one up! All linked now :)
I think we still have too much asymmetry between objects and their descr, but
that can't be helped without exposing the rest to the Lua interface, which I'd
say is out of scope for this branch.
--
https://code.launchpad.net/~widelands-dev/widela
I get it to work, see http://ibb.co/ecziaQ
If you are fine with my wording i will change the other occurrences according
to this example.
To get this working i just used another "role" called :any:. This searches for
all parts wich has a clickable header in the documents. One could use parts of
Continuous integration builds have changed state:
Travis build 2490. State: errored. Details:
https://travis-ci.org/widelands/widelands/builds/259384065.
Appveyor build 2314. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_doc_type_
I spent an hour RTFM and it looks like we can't link to MapObject:descr with
Sphinx. I ran out of time, so I'll have to try again some other time. If
somebody else figures this out in the meantime, it won't hurt my feelings :P
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/
Continuous integration builds have changed state:
Travis build 2488. State: failed. Details:
https://travis-ci.org/widelands/widelands/builds/259145374.
Appveyor build 2313. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_doc_type_n
This looks much better now :-)
One nit: There are now several statements like:
"You can access further properties via its descr."
The word 'its' let me assume that 'descr' is a property of the linked objects,
but there is no such property. Maybe clarify the statement to something like:
"You ca
Review: Resubmit
I have made the documentation more detailed now, added a couple of links and
documented the type hierarchy by linking. Also fixed the bugs that kaputtnik
found.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is
The proposal to merge lp:~widelands-dev/widelands/doc_type_name into
lp:widelands has been updated.
Commit Message changed to:
Fixed and improved documentation for LuaMapObjectDescription::get_type_name.
More cross-linking between map object classes and their description objects.
Fixed some mi
That links to how to define a tribe... it would be more useful to have a
reference to the properties that can be queried at runtime in
https://wl.widelands.org/docs/wl/autogen_wl_map/#module-classes, link to class
wl.map.ImmovableDescription etc.
--
https://code.launchpad.net/~widelands-dev/wid
> We could link them to the LuaDescription objects,
> which already have documentation about he building types etc.
Don't know what you mean here :-S E.g. this page:
https://wl.widelands.org/docs/wl/autogen_toc_lua_tribes_buildings/ ?
Creating a documentation which is useful for beginners is no
We could link them to the LuaDescription objects, which already have
documentation about he building types etc.
No resubmit needed unless you want to send a diff to the mailing list or signal
a big change.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your t
Continuous integration builds have changed state:
Travis build 2476. State: passed. Details:
https://travis-ci.org/widelands/widelands/builds/257274323.
Appveyor build 2301. State: success. Details:
https://ci.appveyor.com/project/widelands-dev/widelands/build/_widelands_dev_widelands_doc_type_n
For me the whole section raises more questions then answers...
It is unclear which string could be used for a query, especially because
'Buildings' and 'Other' are more indented. I guess they are indented because
they are Immovables? So a query for 'building' should first ask for 'immovable'
an
Sorry for the extra commit, using this Ubuntu on Win is a pian.
OTOH I build the shinx Docs now, and found my comment there.
Gun: will we need a resubmint?
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp
Yes, your comment is correct.
We still lack documentation for the world objects, so feel free to add it here.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is subscribed to branch
lp:~widelands-dev/widelands/doc_type_name.
Review: Approve review
Since this is no code change there is no sense in compiling this.
All fine foe me, ther mor docs the better.
If my inline comment abot critter is correct I will iclude it.
Diff comments:
>
> === modified file 'src/scripting/lua_map.cc'
> --- src/scripting/lua_map.cc 201
You're right - I don't know when I gan get to it though.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is requested to review the proposed merge of
lp:~widelands-dev/widelands/doc_type_name into lp:widelands.
___
Since your on it, the documentation in
https://wl.widelands.org/docs/wl/autogen_wl_map/#immovabledescription
looks messed up.
--
https://code.launchpad.net/~widelands-dev/widelands/doc_type_name/+merge/328006
Your team Widelands Developers is requested to review the proposed merge of
lp:~widel
GunChleoc has proposed merging lp:~widelands-dev/widelands/doc_type_name into
lp:widelands.
Commit message:
Fixed and improved documentation for LuaMapObjectDescription::get_type_name.
Requested reviews:
Widelands Developers (widelands-dev)
For more details, see:
https://code.launchpad.net/~w
27 matches
Mail list logo