<div dir="ltr"><div>Hello Stefano,</div><div><br></div><div>I'd suggest going ahead with the merge request. Make sure your changes aren't directly or indirectly responsible for the failing tests beforehand.<br></div><div>(IIRC there aren't any tests for the recent file list.)<br></div><div>Also a separate merge request fixing the failing autotests would be nice.</div><div><br></div><div>Regards,</div><div>Chinmoy<br></div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Fri, Apr 9, 2021 at 8:55 PM Stefano <<a href="mailto:sqrtstefano@gmail.com">sqrtstefano@gmail.com</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"><div dir="ltr">Hi!<br><br>I would like to start contributing to KDE. A few days ago I submitted a bug report for the Okular project (here: <<a href="https://bugs.kde.org/show_bug.cgi?id=435075" target="_blank">https://bugs.kde.org/show_bug.cgi?id=435075</a>>). As I didn't receive any feedback there, and as I had a feeling it was an easy fix, I tried working on it (and it's indeed a few-lines fix).<br><br>According to the guidelines I found here<<a href="https://community.kde.org/Get_Involved/development#Test_your_changes" target="_blank">https://community.kde.org/Get_Involved/development#Test_your_changes</a>>, at this point I should make sure that all unit tests pass before actually trying to submit anything, but I encountered a problem I'm not sure how to deal with: even on the repository code as-is many (19 out of 26) tests fail. So I'm not really sure how to proceed at this point. Should I just submit the merge request? Should I open a bug report for the test suite?<br><br>Best,<br>Stefano</div>
</blockquote></div>