Update clang-tidy rules for Lightning builds under aarch64

[mlxd]

Before submitting

Please complete the following checklist when submitting a PR:

• All new features must include a unit test.
  If you've fixed a bug or added code that should be tested, add a test to the
  tests directory!

• All new functions and code must be clearly commented and documented.
  If you do make documentation changes, make sure that the docs build and
  render correctly by running make docs.

• Ensure that the test suite passes, by running make test.

• Add a new entry to the .github/CHANGELOG.md file, summarizing the
  change, and including a link back to the PR.

• Ensure that code is properly formatted by running make format.

When all the above are checked, delete everything above the dashed
line and fill in the pull request template.

---

Context: More recent versions of clang-tidy are less permission than those used for the general wheel builders. This PR aims to allow us to use clang-tidy-16 on platforms such as aarch64. This will also serve as the base to update the clang-tidy rules to a unified format across all repos in 2024.

Description of the Change: Removes redundant clang-tidy config files, silences unneeded warnings, and makes source changes to allow clang-tidy passes where useful.

Benefits: Better support for clang-tidy across newer platforms.

Possible Drawbacks:

Related GitHub Issues: Also resolves #560

[codecov]

Codecov Report

Attention: 7 lines in your changes are missing coverage. Please review.

Comparison is base (d165de1) 99.02% compared to head (c8628fa) 99.09%.
Report is 1 commits behind head on master.

Files	Patch %	Lines
...rc/simulators/lightning_gpu/StateVectorCudaMPI.hpp	50.00%	2 Missing ⚠️
...imulators/lightning_gpu/StateVectorCudaManaged.hpp	50.00%	2 Missing ⚠️
...lators/lightning_qubit/gates/DynamicDispatcher.hpp	94.44%	2 Missing ⚠️
...ylane_lightning/lightning_qubit/lightning_qubit.py	96.87%	1 Missing ⚠️

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #567      +/-   ##
==========================================
+ Coverage   99.02%   99.09%   +0.06%     
==========================================
  Files         168      201      +33     
  Lines       23235    29203    +5968     
==========================================
+ Hits        23009    28938    +5929     
- Misses        226      265      +39     

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

[vincentmr]

I like all the constant definitions. Looks good to me, thanks @mlxd .

[AmintorDusko]

Hi @mlxd, do you know why we are observing a drop in coverage?

[mlxd]

Hi @mlxd, do you know why we are observing a drop in coverage?

I think this is just due to a reduced line count again --- I don't think any of the test infrastructure changes here, so I'd call it a false flag.

[AmintorDusko]

Thank you for that!

[vincentmr]

Hi @mlxd, do you know why we are observing a drop in coverage?

Given the %'s, I would say it is because parallel-GPU tests are not run.

[AmintorDusko]

[sc-50799]

[AmintorDusko]

Testing (Windows) / C++ tests (Windows) (windows-latest, lightning_qubit) (pull_request) is hanging and failing after the time limit (30 min). I will test again later.