lupyuen commented on issue #19076: URL: https://github.com/apache/nuttx/issues/19076#issuecomment-4686684003
We closed the PR because: 1. NuttX Maintainers feel that embedding PR Metadata (like `depends-on`) inside the PR Body, might confuse PR Authors and PR Reviewers: (1) Editing the PR Metadata won't trigger a rebuild (2) Errors in the PR Metadata are silently ignored 2. NuttX CI Team doesn't have the awk / sed skills to maintain the proposed Shell Script that safely parses and extracts the PR Metadata from the PR Body. Which might lead to Script Injection Attacks in future updates. Perhaps in future, we should create a NuttX Issue first, then assign it to me for discussion, so we can agree on the best solution. -- 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]
