Re: [vim/vim] Refined autoload mechanism explanation (PR #20833)

ubaldot left a comment (vim/vim#20833)

@h-east, I need to be direct here.

I appreciate the formatting suggestions, but I'm concerned about the review process on this PR.

You stated you used an AI tool and "reviewed and verified" the contents, yet your initial analysis contained a fundamental misunderstanding about the difference between lazy loading (52.4) and run-once guards (vim9script noclear). I had to spend significant time explaining this, despite the point being clearly visible at first sight by any human that has some experience with vim script.

Now that the technical discussion is resolved, we're down to line wrapping and spacing. 
While valid, feel like a disproportionate focus given the effort already invested.

The core technical content of this PR is sound. The help text is improved. Can we agree on the substantive changes and handle the remaining formatting as a follow-up, or with a quick final commit from my side?

I'm happy to fix the remaining formatting, but I'd like to avoid another round of AI-generated nitpicks that require manual verification.

Let me know if we can move forward on the substance.

-- 
Reply to this email directly or view it on GitHub:
https://github.com/vim/vim/pull/20833#issuecomment-5091306555
You are receiving this because you are subscribed to this thread.

Message ID: <vim/vim/pull/20833/[email protected]>

-- 
-- 
You received this message from the "vim_dev" maillist.
Do not top-post! Type your reply below the text you are replying to.
For more information, visit http://www.vim.org/maillist.php

--- 
You received this message because you are subscribed to the Google Groups "vim_dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email to [email protected].
To view this discussion visit https://groups.google.com/d/msgid/vim_dev/vim/vim/pull/20833/c5091306555%40github.com.