Make Aqua.test_piracy recursive

[giordano]

Fix #311. Implementation based on NumericalEarth/Breeze.jl#713 and CliMA/Oceananigans.jl#5602.

[codecov]

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 87.50%. Comparing base (3f18a8d) to head (bb5ae5c).
⚠️ Report is 1 commits behind head on master.

Additional details and impacted files

@@            Coverage Diff             @@
##           master     #377      +/-   ##
==========================================
+ Coverage   87.45%   87.50%   +0.04%     
==========================================
  Files          12       12              
  Lines         526      528       +2     
==========================================
+ Hits          460      462       +2     
  Misses         66       66              

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

🚀 New features to boost your workflow:

• ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

[lgoettgens]

Thanks for your PR! Two comments:

1. it unfortunately has conflicts
2. Aqua already has a function walkmodules that could be used here, as it is quite similar. See

   Aqua.jl/src/undocumented_names.jl

   Line 25 in 2f48bcf

   walkmodules(m) do x

   for how it has been used in a different test that needs to recurse submodules

[giordano]

I rebased on master, and used the existing walkmodules, but I removed the recursive argument I initially added to test_piracies because walkmodules makes a lot more annoying to support both a recursive a non-recursive version, and honestly I don't care too much about a non-recursive version (that's a footgun in my opinion)

[giordano]

Bump

[giordano]

Bump