On Sun, Sep 29, 2024 at 9:16 AM Simon Filgis <si...@ingenieurbuero-filgis.de> wrote:
> Hi Mr. Lup, > > I like! > > The LLM should not only check the commenting around the code. > > It should also try to review the code. > 1. Commented and easy to understand? Simplify naming suggestions. > 2. Efficient algorithms? Suggest different implementation, for example > better sorting algorithm. These ^^^ could possibly benefit from LLM, while... 3. Math errors (brackets, casting, rounding, overflow) prevention. > 4. Wrong or missing check of returns detection. These ^^^ could benefit from traditional static analysis tools. I think there are even commercial static analysis tools that provide free checking for open source projects. Perhaps something like that can be hooked up to PRs somehow and automatically post a PR review comment? 5. I guess many more! I definitely like the idea of using technology to assist in PR reviews. Cheers, Nathan