lupyuen commented on PR #13494: URL: https://github.com/apache/nuttx/pull/13494#issuecomment-2360476114
Thanks @cederom! I made some more fixes to our [NuttX PR Bot](https://github.com/lupyuen/nuttx-pr-bot): 1. Check for __Empty Commit Messages__, as requested by @acassis. Our Bot will say _"Please fill in the Commit Message with the PR Summary"_ 1. Remind folks to __Squash Commits__: _"This PR contains 3 Commits. Please Squash the Multiple Commits into a Single Commit"_ 1. __PR Requirements__ for NuttX are now easier to update. [Just send me the patch here](https://github.com/lupyuen/nuttx-pr-bot/blob/main/src/main.rs#L22-L64) 1. Google Gemini API will sometimes fail with HTTP 500 (not sure why). And the Failed Request will be charged to our Daily Quota! So we __retry at most 3 times__, in case the PR has some problematic text that the LLM couldn't handle. 1. How does our Bot know when to stop the retries? It uses [__Emoji Reactions__](https://github.com/lupyuen/nuttx-pr-bot/commit/b71e3f920ffa4a0f08f0cfbe97a46d934c94d04e) to tag each attempt. So we will see the PR tagged with 🚀 which becomes 👀 which becomes 🚀👀 and then the Bot gives up. (If it's hunky dory, our Bot erases its own Emoji Reactions) 1. Our Bot now runs as __GitHub User `nuttxpr`__ that has No Write Access to the NuttX Repo. And I won't get spammed by Every NuttX PR in my email :-) 1. Request by @acassis: Our Bot should check if Kconfig is missing the `---help---` text. Right now our Bot doesn't scan any Source Files, because it will get super complicated and might crash. Wonder if we could extend `nxstyle` to check Kconfig? -- This is an automated message from the Apache Git Service. To respond to the message, please log on to GitHub and use the URL above to go to the specific comment. To unsubscribe, e-mail: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
