You are viewing a plain text version of this content. The canonical link for it is here.

Posted to dev@beam.apache.org by Ning Kang <ni...@google.com> on 2020/08/23 18:13:16 UTC

A whitespacelint has been added to precommit

Hi everyone,

A whitespacelint precommit job has been added in this PR
<https://github.com/apache/beam/pull/12444>.
The job aims to find trailing whitespaces in *non-code* files that are not
scanned by each language specific linter.

Now if your PR contains changes to **.md* or *build.gradle* files, a
`beam_PreCommit_Whitespace` job will be triggered. If the job finds any
whitespace issue, it will report it and list commands to fix it.

To extend the job's coverage to other file types,

   - trigger pattern can be added here
   <https://github.com/apache/beam/blob/master/.test-infra/jenkins/job_PreCommit_Whitespace.groovy#L26>
   - also add the file pattern to be linted here
   <https://github.com/apache/beam/blob/master/sdks/python/test-suites/tox/py38/build.gradle#L44>



Ning.