Skip to content

Commit

Permalink
HBASE-26892 Add spotless:check in our pre commit general check
Browse files Browse the repository at this point in the history
  • Loading branch information
Apache9 committed May 2, 2022
1 parent 9c8c9e7 commit b69e4ef
Showing 1 changed file with 40 additions and 0 deletions.
40 changes: 40 additions & 0 deletions dev-support/hbase-personality.sh
Original file line number Diff line number Diff line change
Expand Up @@ -869,6 +869,46 @@ function hbaseanti_patchfile
return 0
}

######################################

add_test_type spotless

## @description run spotless:check to check format issues
## @audience private
## @stability evolving
## @param repostatus
function spotless_rebuild
{
local repostatus=$1
local logfile="${PATCH_DIR}/${repostatus}-spotless.txt"

if ! verify_needed_test spotless; then
return 0
fi

big_console_header "Checking spotless on ${repostatus}"

start_clock

local -a maven_args=('spotless:check')

# disabled because "maven_executor" needs to return both command and args
# shellcheck disable=2046
echo_and_redirect "${logfile}" $(maven_executor) "${maven_args[@]}"

count=$(${GREP} -c '\[ERROR\]' "${logfile}")
if [[ ${count} -gt 0 ]]; then
add_vote_table -1 spotless "${repostatus} has ${count} errors when running spotless:check, run spotless:apply to fix."
add_footer_table spotless "@@BASE@@/${repostatus}-spotless.txt"
return 1
fi

add_vote_table +1 spotless "${repostatus} has no errors when running spotless:check."
return 0
}

######################################

## @description process the javac output for generating WARNING/ERROR
## @audience private
## @stability evolving
Expand Down

0 comments on commit b69e4ef

Please sign in to comment.