The tone of this warning makes it sound like a bug, and maybe it is, but
it could be a feature too. That and we do not document bugs. Please
rewrite it to simply report the behavior and if this behavior changes in
the future then we will document it.

Warnings were toned done, bugs were noted as features. Goes against my
instinct, though, because in my mind, documentation should be for people
who want to use the extension here and now, and should clearly state
limitations. Nevertheless, hopefully we'll have a SVN 0.3 coming out
soon (I've filed an oodle of bug reports for the extension).

And it's tough to know what behavior will actually change. Also be sure to use &warn.experimental; for this extension in reference.xml as it helps users realize these issues exist. And use &warn.experimental.func; in each function. Related note, one day we should have &warn.experimental; automatically trigger the .func; for every function in the extension. But not today... :)

This has never been done before (that I know of) and although it's
useful I would prefer not having these hard links. We should be able to automate the process of linking constants (livedocs) so please instead
use either <constant> or link to the constants page directly.

I'm not sure what you mean. I changed them to use
svn.constants#svn.constants.auth but that points to a sort of "orphan"
page and I'm not precisely sure what to do.

I meant don't hard link the anchor tags and use something like this:
<link linkend="svn.constants.auth">foo</link>

The link in the output is dealt with by xslt or livedocs.

2. Pick 5 functions

I set the whole set, since they're all complete, but you can review just
the first five if that is easier for you.

Looks good. A few URL entities left and after some configure.xml cleanup it looks commit worthy. Maybe add skeletons for those svn_fs* functions too as a few other functions link (See Also) to them. And for the svn-client-version.xml example remove the reference to PECL4WIN, and in most ref.svn examples use &example.outputs.similar; instead of &example.outputs;. Having fun yet? :)

Regards,
Philip

Reply via email to