Open Source

hideshow 3 child comments

parent

[–] yogthos@lemmy.ml 0 points 1 day ago (2 children)

Can you point to which part specifically are you referring to here? https://github.com/storytold/photocraft

hideshow 2 child comments

parent

[–] bearforcepilot@lemmy.zip 2 points 1 day ago* (1 child)

Almost by definition, maintainability doesn't refer to a "specific part"

https://www.softwareimprovementgroup.com/blog/artcraft-adobe-clone-ai-code-quality/

I'm not interested in getting into an argument about this any further, look through their github issues, there's plenty to learn.

As a note, if you think they're genuinely reviewing this much code at this speed, I think you need to consider the logistics of that.

parent

hideshow 1 child comment

[–] yogthos@lemmy.ml 1 point 1 day ago*

Any project is a sum of its parts, and if you're talking about maintainability, then the parts you could easily point to would be things like high coupling, overly large file sizes, functions that are too big and should be broken up.

Meanwhile, your own link shows that it's roughly within the ballpark of industry average. And if you look through issues for any large project on github you'll see similar types of issues. Given how new the project is, none of this is really surprising or out of the ordinary.

As a note, nowhere did I suggest they do manual code reviews of everything at this speed. What I do think they have is a regression test suite which they build out as issues come up and they fix them.