<div dir="ltr"><div dir="ltr"><br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Aug 20, 2020 at 11:56 AM Dimitris Zacharopoulos (HARICA) <<a href="mailto:dzacharo@harica.gr">dzacharo@harica.gr</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"><br>
I followed <br>
<a href="https://wiki.cabforum.org/github_redline_guide#chair_or_vice_chairactions_after_the_ipr_review_period" rel="noreferrer" target="_blank">https://wiki.cabforum.org/github_redline_guide#chair_or_vice_chairactions_after_the_ipr_review_period</a> <br>
for SC30 and SC31 based on the existing pull request and review <br>
(<a href="https://github.com/cabforum/documents/pull/203" rel="noreferrer" target="_blank">https://github.com/cabforum/documents/pull/203</a>). </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex"> </blockquote><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
The process had a step to delete the temporary branch <br>
(Ballot-SC30-and-31), which I did. Unfortunately, this also closed <br>
<a href="https://github.com/cabforum/documents/pull/207" rel="noreferrer" target="_blank">https://github.com/cabforum/documents/pull/207</a> because it was depending <br>
on the temporary branch Ballot-SC30-and-31.<br>
<br>
Ryan, is this something you can fix? </blockquote><div><br></div><div>Yes, it's no big deal and what I expected to happen.</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">We should probably update the <br>
instructions accordingly.<br></blockquote><div><br></div><div>For what? I used a non-standard approach that intentionally isn't documented to begin with ;-)</div><div><br></div><div>I think we likely want to focus our documentation on common, normal flows intended for those not wanting to learn Git or GitHub (totally understandable) or the Chair and Vice-Chair to produce results, and don't need to try to document every scenario or use case. Weird edge cases (like where I've created multiple branches to show in-progress stuff) is something I think we can and should deal with on an ad-hoc basis, because it's rare, exceptional, and trying to document all the edge cases just introduces more room for failure.</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">
I also went to download the artifacts <br>
(<a href="https://cabforum-travis-artifacts.s3-us-west-2.amazonaws.com/builds/main/BR.docx" rel="noreferrer" target="_blank">https://cabforum-travis-artifacts.s3-us-west-2.amazonaws.com/builds/main/BR.docx</a>) <br>
after the merge, but they were not updated according to the last <br>
commits. I assume that the artifacts are created the first time you <br>
create the pull request and not when you add commits to that pull <br>
request afterwards. Is that correct?<br></blockquote><div><br></div><div>No. The artifacts are generated for every commit.</div><div><br></div><div>It's not clear to me what you're describing, so hopefully you can clarify. The URL you linked is only updated after you merge the commit. If you want to view commits as you add them to the PR, before it is merged, the link is to the branch name - e.g. <a href="https://cabforum-travis-artifacts.s3-us-west-2.amazonaws.com/builds/Ballot-SC30-and-31/BR.docx">https://cabforum-travis-artifacts.s3-us-west-2.amazonaws.com/builds/Ballot-SC30-and-31/BR.docx</a></div><div><br></div><div>If you added commits to the branch after you merged, and did not merge those, don't do that, that's not part of any documented instruction :)</div><div> </div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">
What would be the best way to generate these artifacts now? We probably <br>
need to update the workflow to include steps to create the artifacts <br>
after the last merge.<br></blockquote><div><br></div><div>Doubtful! It's probably just that you didn't let the update process complete :)</div><div><br></div><div><a href="https://travis-ci.org/github/cabforum/documents/builds/719653438">https://travis-ci.org/github/cabforum/documents/builds/719653438</a> shows it took 2 minutes and 6 seconds to generate the artifacts. Did you try to view immediately after committing, and before 2 minutes and 6 seconds had passed?</div><div><br></div><div>This is one of those things that is in the file of "improvements to consider" (switching to <a href="https://github.com/cabforum/documents/projects/1#card-41784176">GitHub actions</a>, so it adds the UI signals). </div></div></div>