<table><tr><td style="">mihac accepted this revision.<br />mihac added a comment.<br />This revision is now accepted and ready to land.
</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/D9306" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>Looks good, thanks for this change. I agree that this plugin should be more in line with the rest of KDevelop.</p>

<p>But regarding multiple commints, from as little as I saw about Phabricator, you can use <tt style="background: #ebebeb; font-size: 13px;">arc</tt> to generate a diff from multiple commits, and paste that diff here. Although I never tried it, doesn't that produce the same result as a single bigger commit? The FindCargo thing is mostly unrelated to the other changes.</p>

<p>The other thing is working with raw diffs is annoying if I want to test the change. In the future, can you just push the branch so I can check it out locally?</p></div></div><br /><div><strong>REPOSITORY</strong><div><div>R894 KDevelop Cargo Support</div></div></div><br /><div><strong>BRANCH</strong><div><div>pimpcmake</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D9306" rel="noreferrer">https://phabricator.kde.org/D9306</a></div></div><br /><div><strong>To: </strong>kossebau, mihac<br /><strong>Cc: </strong>kdevelop-devel<br /></div>