Grouping

Author: jsoref

test-spelling-unknown-words

@@ -86,12 +86,14 @@ xargs_zero() {
     args="$*" "$spellchecker/xargs_zero"
   fi
 }
+echo '::group::Spell check'
 (git 'ls-files' -z 2> /dev/null) |\
   "$spellchecker/exclude.pl" |\
   xargs_zero "$word_splitter" |\
   "$word_splitter" |\
   perl -p -n -e 's/ \(.*//' > "$run_output"
   word_splitter_status="${PIPESTATUS[2]} ${PIPESTATUS[3]}"
+  echo "::endgroup::"
   if [ "$word_splitter_status" != '0 0' ]; then
     echo "$word_splitter failed ($word_splitter_status)"
     exit 2
@@ -202,6 +204,8 @@ bullet_words() {
   fi
   if [ -s "$run_warnings.raw" ]; then
     (
+      echo '::endgroup::'
+      echo '::group::Misspellings'
       echo "::add-matcher::.git/reporter.json"
       cat "$run_warnings.raw"
       echo "::remove-matcher owner=jsoref-spelling::"
@@ -246,6 +250,7 @@ comment() {
 }
 
 if [ ! -e "$whitelist_path" ]; then
+  echo '::group::No whitelist'
   title="No preexisting $whitelist_path file"
   instructions=$(
     echo 'cat > '"$whitelist_path"' <<EOF=EOF'
@@ -254,22 +259,30 @@ if [ ! -e "$whitelist_path" ]; then
     to_publish_whitelist
   )
       spelling_info "$title" "$(bullet_words "$(cat "$run_output")")" "$instructions"
+  echo '::endgroup::'
   quit 2
 fi
+
+echo '::group::Compare whitelist with new output'
 sorted_whitelist="$temp/$(basename $whitelist_path)"
 (sort -u -f "$whitelist_path" | grep . || true) > "$sorted_whitelist"
 whitelist_path="$sorted_whitelist"
 
 diff_output=$(diff -U0 "$whitelist_path" "$run_output" |grep -v "$spellchecker" || true)
+echo '::endgroup::'
 
 if [ -z "$diff_output" ]; then
+  echo '::group::No misspellings'
   title="No new words with misspellings found"
       spelling_info "$title" "There are currently $(wc -l $whitelist_path|sed -e 's/ .*//') whitelisted items." ""
+  echo '::endgroup::'
   quit 0
 fi
 
+echo '::group::New output'
 new_output=$(diff -i -U0 "$whitelist_path" "$run_output" |grep -v "$spellchecker" |\
   perl -n -w -e 'next unless /^\+/; next if /^\+{3} /; s/^.//; print;')
+echo '::endgroup::'
 
 make_instructions() {
   patch_remove=$(echo "$diff_output" | perl -ne 'next unless s/^-([^-])/$1/; print')
@@ -293,16 +306,20 @@ make_instructions() {
 }
 
 if [ -z "$new_output" ]; then
+  echo '::group::Fewer misspellings'
   title='There are now fewer misspellings than before'
   instructions=$(
     make_instructions
   )
       spelling_info "$title" "$(bullet_words "$(new_words "$diff_output")")" "$instructions"
+  echo '::endgroup::'
   quit
 fi
+echo '::group::New misspellings'
 title='New misspellings found, please review'
 instructions=$(
   make_instructions
 )
     spelling_warning "$title" "$(bullet_words "$new_output")" "$instructions"
+echo '::endgroup::'
 quit 1