<table><tr><td style="">dhaumann 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/D10135" rel="noreferrer">View Revision</a></tr></table><br /><div><div><p>I'm late to the game, but in general looks good.</p>

<p>Minor comments:</p>

<ul class="remarkup-list">
<li class="remarkup-list-item">What's missing (or I miss something) is that it could be that entries exist twice, if in resource and on disk. Since this is not the case right now, this is probably not an issue.</li>
<li class="remarkup-list-item">holidays/holidays.qrc is manually maintained and not auto-generated. This is probably fine as well since we don't expect as many changes as e.g. in the syntax-highlighting repo...</li>
</ul></div></div><br /><div><strong>REPOSITORY</strong><div><div>R175 PIM: KHolidays</div></div></div><br /><div><strong>REVISION DETAIL</strong><div><a href="https://phabricator.kde.org/D10135" rel="noreferrer">https://phabricator.kde.org/D10135</a></div></div><br /><div><strong>To: </strong>vkrause, Frameworks, KDE PIM, mlaurent<br /><strong>Cc: </strong>dhaumann, mlaurent, dvasin, winterz, vkrause, knauss, dvratil<br /></div>