[Bug] Fix Swap Hands bug introduced by OSH (qmk#9968)

Fixes the handling for the oneshot cleanup, so it only cleans up if it is active.  It should not cleanup of SHO is off (eg using a normal oneshot key), nor if it's actively pressed or used.  

Previous behavior BROKE swap hand key.

Add PR checklist document. (qmk#9913)

* Add PR checklist document.

* Update docs/pr_checklist.md

Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com>

* Apply suggestions from code review

Co-authored-by: Ryan <fauxpark@gmail.com>

* Reword the lower/raise/adjust suggestion somewhat for clarity.

* Add suggestion from @Didel for coding conventions.

Co-authored-by: James Young <18669334+noroadsleft@users.noreply.github.com>
Co-authored-by: Ryan <fauxpark@gmail.com>

Add Indicator flag for RGB Matrix (qmk#9933)

* Add Indicator flag for RGB Matrix

This adds a new flag for the RGB Matrix feature that lets you specify if the LED is an indicator LED, to be used to indicate the system state of the keyboard (eg caps/num/etc lock status, layer indication, modifer status, etc).

* Better formatting of table

Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866) (qmk#9874)

* Fix RGB_DISABLE_TIMEOUT overflow warning (qmk#9866)

* Adjust capitalization (qmk#9874)

Fix Variable Trace handling (qmk#9831)

It wants a number, but a number of files have it set to "no", even
though it's commented out.  This means that if you set it to no, it
will cause a compiler error.  This sets the default to "no", and
checks to make sure it's not set to "no" before processing it, and
striping the value from it.

format code according to conventions [skip ci]

format code according to conventions [skip ci]

MSYS2: Switch to bootloadHID package and bring back avrdude package (q…

…mk#9736)

* MSYS2: Switch to bootloadHID package and bring back avrdude package

* Update Zadig docs as well

Redefine IS_LAYER_ON/OFF() as aliases for existing layer functions (q…

…mk#6352)

* Add IS_LAYER_ON_STATE()/IS_LAYER_OFF_STATE() macros

* Add docs for IS_LAYER_ON/OFF(_STATE) macros

* Remove IS_LAYER_ON/OFF_STATE redefinition in userspace

* Run clang-format on quantum/quantum.h

* Redefine IS_LAYER_ON/OFF(_STATE) as aliases of existing layer functions

Also update relevant doc entries.

Needs testing to check if this breaks existing IS_LAYER_ON/OFF usage in certain
edge cases (namely calling the macros with 0).

* Reformat layer check function docs

Fixed CA_DOTA key code (qmk#9722)