On August 9, 2019 2:04:32 PM Glen Fernandes via Boost
On Friday, August 9, 2019, Cromwell Enage wrote:
Is there time to merge Boost.Parameter PR #90, a documentation change?
Looks like PR #90 removes ALL the documentation sources with a commit message "Temporarily remove documentation until they are fixed...".
Why _temporarily_ remove anything? Just disable the relevant step in doc/Jamfile? Before shipping a Boost.Parameter in a Boost release, have at least the fixed documentation sources int here, or the existing ones.
There's no value in PR90 going into Boost 1.71 for users is there?
If the current docs are completely wrong then I'd say revert the code to correspond to docs, at least remotely. The new code should be considered not ready for release in this case. I doubt though that the old docs are completely useless. I don't think we had to completely rewrite Boost.Parameter-related code in other libraries, which means the old interfaces and their docs are still relevant. In any case, releasing a library with no docs is not acceptable.