[
https://issues.apache.org/jira/browse/SPARK-8722?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=14606925#comment-14606925
]
Josh Rosen commented on SPARK-8722:
-----------------------------------
If we wanted to get _really_ fancy, we could warn whenever the most recent test
run was prior to the most recent style-checker / linter change, which would
help to avoid the cluster of build breaks that happens whenever we change the
style checker rules.
> PR merge script should warn when merging a PR that has failed tests
> -------------------------------------------------------------------
>
> Key: SPARK-8722
> URL: https://issues.apache.org/jira/browse/SPARK-8722
> Project: Spark
> Issue Type: New Feature
> Components: Project Infra
> Reporter: Josh Rosen
>
> It would be helpful if our PR merge script looked at the GitHub comments and
> warned the committer if the PR being merged had failed its most recent test
> run. It might also be nice to log a warning if the PR was last tested more
> than X days ago, since a PR that hasn't recently been tested might be failing
> new style checks or other tests. We should provide an option to continue and
> force-merge despite the test failure.
--
This message was sent by Atlassian JIRA
(v6.3.4#6332)
---------------------------------------------------------------------
To unsubscribe, e-mail: [email protected]
For additional commands, e-mail: [email protected]
