modularity / fedmod

#91 Improve summarize-module tool
Closed by nphilipp. Opened by rdossant.
modularity/ rdossant/fedmod summarize-module  into  master

Download 91.patch
  • Add context and version information
  • Add modular dependencies information
  • Add option to show table as a tree

Metadata Update from @nphilipp:
- Request assigned

Unfortunately, the test suite doesn't pass here. I've checked and commit da59d30 "mark print_summary as internal" is the first one that fails.

Since I fixed the last remaining PEP8 errors in the master branch, I want that both the flake8 tool and the test suite always run clean (i.e. on every commit), rather than have one commit in a PR introduce a bug which is fixed by a subsequent commit—this makes bisecting issues easier later on. If you want, I can fix the commits myself, I'd rebase onto the current master anyway. :wink:

rebased onto 3fe7bcde9089b25e16890ca0c5fd4e836ba62baa

Fixed and rebased.

6 new commits added

  • summarize-module: add modular dependency info
  • summarize-module: add version and context to output
  • Flatten the module profiles lookup dict
  • summarize-module: make shallow copy of lookup dicts
  • summarize-module: mark print_summary as internal
  • Revert "repodata: make get_dataset public"

Tests fixed.

Thanks for fixing the tests! I've applied your changes in commits c727c16..6add261, with some small changes regarding imports which flake8 flagged.

Pull-Request has been closed by nphilipp
Metadata
nphilipp
No Tags
Static archive. pagure.io was sunset in 2026; this is a read-only snapshot.
Red Hat Logo

Fedora is sponsored by Red Hat. Learn more about the relationship between Red Hat and Fedora »

© Red Hat, Inc. and others.