Hi Stefan,

I took gallia to review. Thanks for the work. I leave you some comments for now:

- d/upstream/metadata: Changelog get 404
- d/upstream/metadata: Documentation get 404
- d/control: You can set all B-depends used *only* in tests with <!nocheck>.
- d/control: Homepage seems to be incorrect.
- d/copyright: Please use Expat instead of MIT.
- tests/bats/testfiles/log-01.json.zst: This file is necessary to work? or it's
  just used in test? The test is important?
- Please review d/copyright file, for instance file
  src/gallia/pydantic_argparse/utils/field.py is Apache-2.0, but in d/copyright
  says MIT.

I stop review it here. Please let me know when you have a new version to review.

Thanks!

-- 
cheers,
        Emmanuel Arias

 ⢀⣴⠾⠻⢶⣦⠀
 ⣾⠁⢠⠒⠀⣿⡁  [email protected]
 ⢿⡄⠘⠷⠚⠋⠀  OpenPGP: 13796755BBC72BB8ABE2AEB5 FA9DEC5DE11C63F1
 ⠈⠳⣄

Attachment: signature.asc
Description: PGP signature

Reply via email to