Ignore notebooks when checking for newlines (#61156)

driazati · facebook-github-bot · commit 6e9e30cc1d70 · 2021-07-01T18:11:43.000-07:00
Summary: Fix lint on master (these files should be considered "generated" so don't lint them) Pull Request resolved: #61156 Reviewed By: malfet Differential Revision: D29532211 Pulled By: driazati fbshipit-source-id: a1e47f45bedf441613bdc2bd60fbf8299e5c962f
diff --git a/.github/workflows/lint.yml b/.github/workflows/lint.yml
@@ -31,7 +31,7 @@ jobs:
       - name: Ensure correct trailing newlines
         if: always() && steps.requirements.outcome == 'success'
         run: |
-          (! git --no-pager grep -Il '' -- . ':(exclude)**/contrib/**' ':(exclude)third_party' ':(exclude)**.expect' ':(exclude)tools/clang_format_hash' | tools/linter/trailing_newlines.py || (echo "The above files do not have correct trailing newlines; please normalize them"; false))
+          (! git --no-pager grep -Il '' -- . ':(exclude)**/contrib/**' ':(exclude)third_party' ':(exclude)**.expect' ':(exclude)**.ipynb' ':(exclude)tools/clang_format_hash' | tools/linter/trailing_newlines.py || (echo "The above files do not have correct trailing newlines; please normalize them"; false))
       - name: Ensure no trailing spaces
         if: always()
         run: |
