• catlover@sh.itjust.worksOP
    link
    fedilink
    arrow-up
    5
    arrow-down
    3
    ·
    2 years ago

    I can totally understand the lack of people working on the project. As skeptic I am, I must say that you have total right to not believe me, but I’m an open source developer too with an own project, with literally no time at all since years. But what I’m trying to say: if the devs have no time to review PR-s, and features, please just don’t merge them at all. Please tell me that I’m wrong, but this seems like PR-s getting merged without proper reviews. This is far more worse than not merging any PR-s

    • Dandroid@dandroid.app
      link
      fedilink
      arrow-up
      0
      ·
      2 years ago

      I haven’t looked through the Jerboa codebase very closely yet (I downloaded it, but that’s about as far as I have got so far), but it sounds like they either need more code coverage for their unit tests or they need to improve the quality of the unit tests. Good unit tests should catch a lot of bugs introduced by PRs. There is no QA team to do regression testing for each release, and the main developers can’t be expected to run manual testing for every PR. The developer of the PR should be doing that, but clearly they are not.