[
https://issues.apache.org/jira/browse/IMPALA-7317?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=16563204#comment-16563204
]
ASF subversion and git services commented on IMPALA-7317:
---------------------------------------------------------
Commit 47b4606742287361b8c3751b4db77ab5b3508783 in impala's branch
refs/heads/master from [[email protected]]
[ https://git-wip-us.apache.org/repos/asf?p=impala.git;h=47b4606 ]
IMPALA-7317: add scripts to post flake8 comments
The script is used by a new jenkins job gerrit-auto-critic
to post comments on code reviews.
Testing:
This patch deliberately contains some flake8 violations so
that gerrit-auto-critic will flag them.
Change-Id: I7d348ea4944f829a407bd7b2939654f272736170
Reviewed-on: http://gerrit.cloudera.org:8080/11054
Reviewed-by: Impala Public Jenkins <[email protected]>
Reviewed-by: Michael Brown <[email protected]>
Tested-by: Impala Public Jenkins <[email protected]>
> Instant validation job that runs on Gerrit upload (clang tidy, other simple
> checks)
> -----------------------------------------------------------------------------------
>
> Key: IMPALA-7317
> URL: https://issues.apache.org/jira/browse/IMPALA-7317
> Project: IMPALA
> Issue Type: Task
> Reporter: Tim Armstrong
> Assignee: Tim Armstrong
> Priority: Major
>
> Add a new job to jenkins.impala.io that runs clang tidy upon upload to
> Gerrit. The goal is to prefetch any obvious failures that will prevent the
> code being merged.
> Existing checks that are reasonably cheap:
> * clang-tidy-ub1604
> * ubuntu-16.04-build-only
> * rat-check-ub1604
> Look into other checks to add (flake8, lines too long, etc)
--
This message was sent by Atlassian JIRA
(v7.6.3#76005)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
