Skip to content

Conversation

lirshindalman
Copy link
Contributor

@lirshindalman lirshindalman commented Oct 5, 2025

By submitting this pull request, I confirm that my contribution is made under the terms of the Apache 2.0 license.

Description

Fixed a bug where the link between resources was incorrect when multiple nested modules were involved.
Updated the function to compute the longest common prefix between two module names by comparing all dot-separated parts instead of only the first two segments.

Fixes # (issue)
link between resources was incorrect when multiple nested modules were involved

New/Edited policies (Delete if not relevant)

Description

Include a description of what makes it a violation and any relevant external links.

Fix

How does someone fix the issue in code and/or in runtime?

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have made corresponding changes to the documentation
  • I have added tests that prove my feature, policy, or fix is effective and works
  • New and existing tests pass locally with my changes

@maxamel
Copy link
Contributor

maxamel commented Oct 8, 2025

Is it possible to add a test for the handled use-case ?

@lirshindalman lirshindalman merged commit ac92373 into main Oct 8, 2025
46 checks passed
@lirshindalman lirshindalman deleted the fix_find_vertex_with_best_match branch October 8, 2025 15:29
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