<div dir="ltr"><div>Hi Harrissou,</div><div><br></div><div>No worries, I have been more focused on technical matters up to now, but I was indeed expecting some room for improvements. There was a bit of a stress to provide reports for the grant proposal, but it was clearly a call for discussion, sorry not to state it clearly.</div><div><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">Le mar. 7 janv. 2020 à 02:59, DelazJ <<a href="mailto:delazj@gmail.com">delazj@gmail.com</a>> a écrit :<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Hi all,</div><div><br></div><div>Thanks Denis for the work.</div><div>I might be missing some key points because comparing the generated reports from the two systems, I'm sorry I feel like it's instead a regression. Alow me to explain:</div><div><br></div><div>1. For the same feature merged in the code, see old system report [0] vs new system's [1]. From a doc writer perspective, I get more information from the first one than the second.</div></div></blockquote><div><br></div><div>I see. So following Nyall's suggestion:</div><div>1. Copy description</div><div>2. Copy commit message of commits having [needs-doc] (similarly to now)</div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>2. Another point is that milestone is what we use to filter issues reports and manage the docs schedule, so if it's not set by the developer (assuming that the dev knows the milestone to indicate), someone has to do it manually in the generated report. With the current system, when we enter a new development cycle, we (Richard and myself) set the new milestone (for LTR) and the target version label [2] and then, every generated report is automatically filled with these information at their creation. Done once and nobody cares about anymore. Until the next release.</div><div>This new system means devs "should" enter that information for each doc-related PR. I can't count the number of times I made a remind for the [needs-docs] label, and the PR was merged without...</div></div></blockquote><div><br></div><div>I see.</div><div>So, you take the assumption that any request for documentation happens in the current release and not in the past.</div><div>I guess it might make sense and that it would still be possible to handle manually the case.</div><div>I will see if I can do some nice grepping of the CMakeLists (or as Matthias suggested max(release-x_y)+2 ) to calculate labels and milestone.<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div><br></div><div>3. What is meant by "developers should take care of it"? When/where will the details of the feature be available? If the dev wants to write about his changes in our docs, OK. Otherwise, are we not overloading their workload while they could have provided the necessary bits in the commit message, as they should be doing currently. <br></div><div>What I understood from the proposal is that developers will be encouraged to detail their feature in the PR message, the place they sell their feature to others, using a simple and accessible language. And then, at the merge time, the message of the PR (with maybe screenshots) will be copied to the generated report in docs, allowing writers to see what the feature is. Did I misunderstand or have the options changed meanwhile?</div></div></blockquote><div><br></div><div>I will update the message explaining how the bot works and that the PR author should document in the description. <br></div><div>And there, if you have any improvement to make as text changes feel free to go ahead.</div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><div dir="ltr"><div>Denis</div></div>
</blockquote></div></div>