[BUG] Fix Key Override includes

[drashna]

Description

Include was wrong. Fixed.

Types of Changes

• Core
• Bugfix

Issues Fixed or Closed by This PR

• Discord reported

Checklist

• My code follows the code style of this project: C, Python
• I have read the PR Checklist document and have made the appropriate changes.
• My change requires a change to the documentation.
• I have updated the documentation accordingly.
• I have read the CONTRIBUTING document.
• I have added tests to cover my changes.
• I have tested the changes and verified that they work and don't break anything (as well as I can manage).

[zvecr]

I would rather keep the includes the same, and just do the following,

diff --git a/quantum/process_keycode/process_key_override.c b/quantum/process_keycode/process_key_override.c
index 8b45a9404..09b272507 100644
--- a/quantum/process_keycode/process_key_override.c
+++ b/quantum/process_keycode/process_key_override.c
@@ -18,7 +18,7 @@
 #include "quantum.h"
 #include "report.h"
 #include "timer.h"
-#include "process_key_override_private.h"
+#include "process_key_override.h"
 
 #include <debug.h>
 

[drashna]

I can do so, but is there a specific reason for doing so?

quantum.h contains all of other includes, and this does need a good number of includes.

Also, just this would work too:

#include "process_key_override.h"
#include "quantum.h"

[zvecr]

Including the bare minimum is general best practice, and things like quantum.h including the world produces many assumptions that are harder to unpick later. As the consumer of the api, why would you need to care about another enabled feature like maybe audio.