https://github.com/kiranchandramohan approved this pull request.
> > If there is another way to provide the same reminder when a test is added, > > or if integration tests are no more a concern for llvm, or if everyone > > feels that duplicating the message is not serving the purpose then we can > > consider switching. > > Maybe an RFC to decide on this issue would be a good idea, since removing all > these existing comments (if that's what people agree on) can be done > separately afterwards at very little cost. I don't think that this decision > should be a blocker for this patch. Agree that this PR need not be blocked waiting for a decision. I have started a discourse [thread](https://discourse.llvm.org/t/removing-or-reducing-redundant-messages-in-integration-tests/91591) to discuss this. Given that we have a plan to address this issue, @arsenm would it be OK to submit this PR and follow up with a resolution in a later PR? https://github.com/llvm/llvm-project/pull/215536 _______________________________________________ llvm-branch-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-branch-commits
