You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@superset.apache.org by cr...@apache.org on 2023/10/13 18:00:15 UTC

[superset] branch master updated: build(ci): Provide diff for pre-commit failures (#25638)

This is an automated email from the ASF dual-hosted git repository.

craigrueda pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/superset.git


The following commit(s) were added to refs/heads/master by this push:
     new 5054b9fade build(ci): Provide diff for pre-commit failures (#25638)
5054b9fade is described below

commit 5054b9fadeea28885526861e3c7db5ed0e36464b
Author: Josh Soref <21...@users.noreply.github.com>
AuthorDate: Fri Oct 13 14:00:09 2023 -0400

    build(ci): Provide diff for pre-commit failures (#25638)
    
    Signed-off-by: Josh Soref <21...@users.noreply.github.com>
---
 .github/workflows/superset-python-misc.yml | 7 ++++++-
 1 file changed, 6 insertions(+), 1 deletion(-)

diff --git a/.github/workflows/superset-python-misc.yml b/.github/workflows/superset-python-misc.yml
index a507467cad..e42c743c9e 100644
--- a/.github/workflows/superset-python-misc.yml
+++ b/.github/workflows/superset-python-misc.yml
@@ -86,7 +86,12 @@ jobs:
           echo "HOMEBREW_REPOSITORY=$HOMEBREW_REPOSITORY" >>"${GITHUB_ENV}"
           brew install norwoodj/tap/helm-docs
       - name: pre-commit
-        run: pre-commit run --all-files
+        run: |
+          if ! pre-commit run --all-files; then
+            git status
+            git diff
+            exit 1
+          fi
 
   babel-extract:
     runs-on: ubuntu-20.04