<table><tr><td style="">rkflx added a comment.
</td><a style="text-decoration: none; padding: 4px 8px; margin: 0 8px 8px; float: right; color: #464C5C; font-weight: bold; border-radius: 3px; background-color: #F7F7F9; background-image: linear-gradient(to bottom,#fff,#f1f0f1); display: inline-block; border: 1px solid rgba(71,87,120,.2);" href="https://phabricator.kde.org/D11470">View Revision</a></tr></table><br /><div><div><p>Sorry for chiming in, but in light of the focus goal <a href="https://phabricator.kde.org/T7116" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">T7116: Streamlined onboarding of new contributors</a> it's important to fix rough edges in our infrastructure, too:</p>

<blockquote style="border-left: 3px solid #8C98B8;
          color: #6B748C;
          font-style: italic;
          margin: 4px 0 12px 0;
          padding: 8px 12px;
          background-color: #F8F9FC;">
<div style="font-style: normal;
          padding-bottom: 4px;">In <a href="https://phabricator.kde.org/D11470#252163" style="background-color: #e7e7e7;
          border-color: #e7e7e7;
          border-radius: 3px;
          padding: 0 4px;
          font-weight: bold;
          color: black;text-decoration: none;">D11470#252163</a>, <a href="https://phabricator.kde.org/p/jpoelen/" style="
              border-color: #f1f7ff;
              color: #19558d;
              background-color: #f1f7ff;
                border: 1px solid transparent;
                border-radius: 3px;
                font-weight: bold;
                padding: 0 4px;">@jpoelen</a> wrote:</div>
<div style="margin: 0;
          padding: 0;
          border: 0;
          color: rgb(107, 116, 140);"><p>I am completely lost with <tt style="background: #ebebeb; font-size: 13px;">arc diff</tt>. The command adds files that I did not normally modify in this branch and specifiying a reference commit does not seem to work.</p></div>
</blockquote>

<p>Do you know about our recommended workflow in <a href="https://community.kde.org/Infrastructure/Phabricator#Workflow" class="remarkup-link" target="_blank" rel="noreferrer">https://community.kde.org/Infrastructure/Phabricator#Workflow</a>? Happy to help with that, just let me know.</p>

<blockquote style="border-left: 3px solid #a7b5bf; color: #464c5c; font-style: italic; margin: 4px 0 12px 0; padding: 4px 12px; background-color: #f8f9fc;"><p>I ended up doing everything through a new branch, sorry to rot the history like this :/.</p></blockquote>

<p>Phabricator's Differential is simply a list of patch iterations if you will, there is no "history" as such which you might have impacted. Only if you use the web uploader instead of Arcanist you make life for the reviewer a bit harder (no author information when landing, no context in the diff view, no base commit).</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R216 Syntax Highlighting</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D11470">https://phabricator.kde.org/D11470</a></div></div><br /><div><strong>To: </strong>jpoelen, Framework: Syntax Highlighting, dhaumann<br /><strong>Cc: </strong>rkflx, Frameworks, michaelh, bruns<br /></div>