search

cta-observatory / ctapipe

Low-level data processing pipeline software for CTAO or similar arrays of Imaging Atmospheric Cherenkov Telescopes
https://ctapipe.readthedocs.org
BSD 3-Clause "New" or "Revised" License
63 stars 266 forks source link

Switch to src based repository structure #2459

Closed maxnoe closed 6 months ago

maxnoe commented 8 months ago

Immediate issue:

Since all files were moved, they were all "touched" in this PR, which triggered the pre-commit hook to reformat all files.

So they are not just moved but also reformatted.

maxnoe commented 8 months ago

But actually the changes are very small, so maybe not that large of an issue

codecov[bot] commented 8 months ago

Codecov Report

All modified and coverable lines are covered by tests :white_check_mark:

Comparison is base (d0ed459) 92.52% compared to head (5d2aa9e) 92.52%.

Additional details and impacted files ```diff @@ Coverage Diff @@ ## main #2459 +/- ## ========================================== - Coverage 92.52% 92.52% -0.01% ========================================== Files 234 234 Lines 20006 20005 -1 ========================================== - Hits 18511 18510 -1 Misses 1495 1495 ```

:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.

maxnoe commented 6 months ago

Missing a changelog, and I don't feel able to review the dark magic in setup.cfg, otherwise it looks fine

See setuptools documentation: https://setuptools.pypa.io/en/latest/userguide/package_discovery.html#package-discovery-and-namespace-packages