Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

[Gradle] Correctly identify all modules' group and version #1365

Merged
merged 1 commit into from
Sep 9, 2024
Merged

[Gradle] Correctly identify all modules' group and version #1365

merged 1 commit into from
Sep 9, 2024

Conversation

malice00
Copy link
Contributor

@malice00 malice00 commented Sep 9, 2024

We build apps using Expo, which adds several modules to our build. Currently, these modules are added several times as components with different groups and versions. This PR addresses this by loading the correct data for all modules and caching it to use whenever a module is referenced instead of relying on incorrect defaults.
This PR is based on #1337, which must therefore be resolved before this can be merged.

@malice00 malice00 requested a review from prabhu as a code owner September 9, 2024 01:15
@malice00 malice00 changed the title [Gradle] Correctly identify module group and version [Gradle] Correctly identify all modules' group and version Sep 9, 2024
@prabhu
Copy link
Collaborator

prabhu commented Sep 9, 2024

@malice00 have merged #1337. Could you kindly rebase and take a look at the conflicts?

Signed-off-by: Roland Asmann <roland.asmann@gmail.com>
@malice00
Copy link
Contributor Author

malice00 commented Sep 9, 2024

@prabhu Sure, no problem!

Copy link
Collaborator

@prabhu prabhu left a comment

Choose a reason for hiding this comment

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

Looks great! Thank you so much!

@prabhu prabhu merged commit b61f589 into CycloneDX:master Sep 9, 2024
18 of 21 checks passed
@malice00 malice00 deleted the fix/component_id branch September 9, 2024 18:26
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants