Skip to content

Conversation

@alexandregaldeano
Copy link

@alexandregaldeano alexandregaldeano commented Dec 6, 2024

In our codebase we needed to know which modules were the most depended on.

So, here it is, the inverse dependency tree, where:

account (13.0+c)
└── c (no version)
    └── a (13.0.1.0.0)

Reads "account is a dependence of c, and c is a dependence of a".

Changes made:

  • Added the --inverse option to the tree command
  • Added the parents field in the tree.Node class, which is populated in the tree.tree_command.add function
  • The parents field is used instead of the children field in the tree.Node.print method when in inverse mode

@alexandregaldeano alexandregaldeano marked this pull request as ready for review December 6, 2024 15:24
Copy link

@petrus-v petrus-v left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Nice one ;)

Technically Code looks good to me !

@sbidoul
Copy link
Member

sbidoul commented Sep 22, 2025

Thanks @alexandregaldeano. Can you rebase and add a news/75.feature file with a short changelog entry?

I plan a release soon to fix Odoo 19 support (#80).

@alexandregaldeano
Copy link
Author

@sbidoul noted, I try to do it by the end of the week 😃

@alexandregaldeano alexandregaldeano force-pushed the agaldeano/inverse-tree branch 2 times, most recently from 6112533 to 8fd7561 Compare September 25, 2025 14:09
@alexandregaldeano
Copy link
Author

@sbidoul I updated my branch foodles-tech:agaldeano/inverse-tree to acsone:main instead of updating foodles-tech:main which had the effect to drop my commit and close this PR 😬
Could you please reapprove the test workflow? 🙏

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants