kezhenxu94 commented on pull request #6748: URL: https://github.com/apache/skywalking/pull/6748#issuecomment-819163335
> > Adding component id is common case so I added the file into skip list > > The side effect is, if a PR breaks the YAML format, we will face all tests failure on all next PRs :P Yes indeed, this is a tradeoff to reduce the CI resource load while increasing the reviewers' attention, since it's skipped when only `component-libraries.yml` is modified, it's acceptable to review its format as well, let's keep this until we really encounter the case you said. -- 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. For queries about this service, please contact Infrastructure at: us...@infra.apache.org
