Zikun Zhu

Results 23 comments of Zikun Zhu

> The changes so far are looking good. How much more work do you estimate it'll take to get to a "ready" state? > > See if you can get...

> > The changes so far are looking good. How much more work do you estimate it'll take to get to a "ready" state? > > See if you can...

All checks have passed and no more trag-bot comments for the time being, I will proceed to address the remaining comments @subhramit raised during code review. Please feel free to...

> @zikunz note that not all tragbot comments are relevant. E.g. if you are dealing with pure bollean functions, assertTrue and assertFalse is alright to use. Thank you for your...

> Could you also answer [#12912 (comment)](https://github.com/JabRef/jabref/pull/12912#discussion_r2052392686) Yes sure, please give me some time, I will quote the comment and answer it :)

> > Could you also answer [#12912 (comment)](https://github.com/JabRef/jabref/pull/12912#discussion_r2052392686) > > Yes sure, please give me some time, I will quote the comment and answer it :) Thank you for raising...

> Hi, @zikunz! I think I can hop in and help Subhramit et al. with this PR. > > As JabRef is primarily done in free time by volunteers, we...

> Please not another Manager. Especially not a repository manager. This is silly, almost funny. The repository provides everything the application needs. If it needs to be refreshed because preferences...

> > I think I complicated things unnecessarily by creating a separate manager. When you said "the repository provides everything the application needs," it then clicked that I should have...

> Your code currently does not meet [JabRef's code guidelines](https://devdocs.jabref.org/getting-into-the-code/guidelines-for-setting-up-a-local-workspace/intellij-13-code-style.html). We use [Checkstyle](https://checkstyle.sourceforge.io/) to identify issues. You can see which checks are failing by locating the box "Some checks were...