refactor: Remove unused validation includes #23044

1. 
   MarcoFalke commented at 10:18 am on September 20, 2021: member

   Unused includes will cause needless recompilation when headers are changed. Also, they pretend there are dependencies that don’t exist.

   Fix both by removing them.

2. refactor: Remove unused validation includes fa45a1338a
3. 
   MarcoFalke commented at 10:20 am on September 20, 2021: member

   Done after #22626#pullrequestreview-745479829
4. fanquake added the label Refactoring on Sep 20, 2021
5. 
   laanwj commented at 12:41 pm on September 20, 2021: member

   Code review ACK fa45a1338adb127d69aee982920e29519bc1fed6
6. 
   DrahtBot commented at 5:21 am on September 21, 2021: member

   The following sections might be updated with supplementary metadata relevant to reviewers and maintainers.

   Conflicts

   Reviewers, this pull request conflicts with the following ones:

   • #22951 (consensus: move amount.h into consensus by fanquake)
   • #10443 (Add fee_est tool for debugging fee estimation code by ryanofsky)

   If you consider this pull request important, please also help to review the conflicting pull requests. Ideally, start with the one that should be merged first.

7. theStack approved
8. 
   theStack commented at 10:34 am on September 22, 2021: member

   ACK fa45a1338adb127d69aee982920e29519bc1fed6 ♻️
9. MarcoFalke merged this on Sep 22, 2021
10. MarcoFalke closed this on Sep 22, 2021

    ---

11. MarcoFalke deleted the branch on Sep 22, 2021
12. sidhujag referenced this in commit 3f7b0cd268 on Sep 22, 2021
13. DrahtBot locked this on Oct 30, 2022

Contributors
MarcoFalke laanwj DrahtBot theStack

Labels
Refactoring