[ https://issues.apache.org/jira/browse/FLINK-37534?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=17937903#comment-17937903 ]
Thomas Cooper commented on FLINK-37534: --------------------------------------- That is a good point, for the 1.x branches we could simply change the default value in the `workflow_call` trigger to 11. > Add Liniting checks to all PRs > ------------------------------- > > Key: FLINK-37534 > URL: https://issues.apache.org/jira/browse/FLINK-37534 > Project: Flink > Issue Type: Improvement > Reporter: Thomas Cooper > Priority: Not a Priority > > Often when a user submits a PR they will wait for the CI to run in Azure > DevOps and this will fail as they have forgotten to run spotless or > checkstyle. In order to find this out, the author has to dig through the > Azure DevOps UI and often parse the logs. > We currently have a GitHub Action for running "pre-compile" checks that > include all the standard linting checks: > `.github/workflows/template.pre-compile-checks.yml` > It would be a better experience for developers if this was run for every PR > (using the default JVM, currently 17). That way a clear indication would be > given, via the GitHub UI, if the PR has passed the basic checks. -- This message was sent by Atlassian Jira (v8.20.10#820010)