linguini1 commented on PR #3397: URL: https://github.com/apache/nuttx-apps/pull/3397#issuecomment-3843187840
> @linguini1 wants rwsem test data. I used an ai tool to generate a relatively complete test case for testing. I think there's no problem with this. And @linguini1 wants me to share the test case with him, so I uploaded it here to give him a link. I didn't want to incorporate this pr into nuttx-apps. I think there's nothing wrong with using ai appropriately. What's wrong is that I shouldn't have pushed it here. It should only be pushed to my own homepage，so I closed this PR. You're right, thank you for providing the code you used to test. I did request on another PR (don't have the link off-hand) that you share the tests case you used to verify your change. I appreciate you sharing that information with me, AI-generated or not, because it gives me more information about your testing. The reason I reviewed this negatively is as you said, it shouldn't be merged into NuttX. I appreciate you wanting to upstream your testing, but there is currently [an ongoing discussion about admitting AI code into NuttX](https://www.mail-archive.com/[email protected]/msg14173.html). Currently my stance is no. If you have a different opinion, please share it in the discussion on the mailing list so we can make an informed decision when/if we vote on it! > BTW, please refuse the unreasonable requirement from @linguini1 in the future. If you want the new test, please develop it by yourself @linguini1. Appropriate PR testing information is a requirement for contributions to NuttX, in the contributing guidelines and voted for in the mailing least. I don't know why this is an unreasonable request. I will not be writing test cases for everyone's PRs: if you want it merged, prove that it works. -- 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]
