lordgamez commented on code in PR #2010:
URL: https://github.com/apache/nifi-minifi-cpp/pull/2010#discussion_r2278473364
##########
.github/workflows/ci.yml:
##########
@@ -535,32 +535,18 @@ jobs:
with:
name: behavex_output
path: build/behavex_output
- cpplint:
- name: "cpplint"
+ linters:
Review Comment:
I don't think it's a good idea to merge the linters in a single job, it's
better to see the results of them separately. If multiple of them fails only 1
CI run is needed to see all of them, otherwise you will only discover the
second failure in the second CI run after fixing the first issue.
--
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]
