linguini1 commented on PR #18519:
URL: https://github.com/apache/nuttx/pull/18519#issuecomment-4031000570

   > Fair point. The public repo is new but the tool has been in development 
for a bit longer internally. Its value is specifically in automated power-loss 
testing during OTA updates, which as far as I know no other tool covers. The 
checkout is pinned by full commit SHA so the workflow can't be affected by 
later changes to the repo. But I understand the concern about depending on an 
external project. Happy to run this on my own fork instead if that's preferred, 
or open to other suggestions for how to integrate this kind of testing.
   
   Thanks for addressing my concerns! This seems pretty reasonable. The reason 
I mentioned it is because there were similar concerns about 
[NTFC](https://nuttx.apache.org/docs/latest/testing/ntfc.html), but 
repositories have since been made for it. Maybe eventually you might consider 
adding some minimal version to the upstream? Anyways, thanks for your hard 
work! :)


-- 
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]

Reply via email to