test: Bump shellcheck version to 0.8.0 #23635

1. 
   hebasto commented at 1:57 PM on November 30, 2021: member

   Among added rules, SC2295 could be useful for us.

2. DrahtBot added the label Tests on Nov 30, 2021
3. 
   laanwj commented at 6:58 PM on November 30, 2021: member

   Code review ACK 5f27645f2fe6314973e3cc12cc6d8de6a492a520

4. 
   hebasto commented at 7:07 PM on November 30, 2021: member

   Going to rebase on top of #23506.

5. test: Bump shellcheck version to 0.8.0 5202bd1dc0
6. hebasto force-pushed on Nov 30, 2021
7. 
   hebasto commented at 7:18 PM on November 30, 2021: member

   Rebased 5f27645f2fe6314973e3cc12cc6d8de6a492a520 -> 5202bd1dc0942953c415ef39c14ffd48cbaab6b9 (pr23635.01 -> pr23635.02) on top of the #23506.

8. MarcoFalke requested review from dongcarl on Nov 30, 2021
9. 
   dongcarl commented at 7:37 PM on November 30, 2021: member

   Code Review ACK 5202bd1dc0942953c415ef39c14ffd48cbaab6b9

   SC2295 seems like a subtle thing that could've drained valuable man-hours, good to have it checked!

10. MarcoFalke requested review from fanquake on Nov 30, 2021
11. fanquake approved
12. 
    fanquake commented at 2:44 AM on December 1, 2021: member

    ACK 5202bd1dc0942953c415ef39c14ffd48cbaab6b9 - would have rather this just been a part of #23506 to avoid another PR and pointless rebasing.

13. fanquake merged this on Dec 1, 2021
14. fanquake closed this on Dec 1, 2021

    ---

15. sidhujag referenced this in commit f566edbe76 on Dec 1, 2021
16. hebasto deleted the branch on Dec 1, 2021
17. RandyMcMillan referenced this in commit c3d6d3c7d1 on Dec 23, 2021
18. DrahtBot locked this on Dec 1, 2022

Review Requested
dongcarl