-
-
Notifications
You must be signed in to change notification settings - Fork 86
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
fix: Add additional log context when reference can not be found #662
fix: Add additional log context when reference can not be found #662
Conversation
Any fix for this one? |
Hello shawner18, I'd like to merge this, but the sign-off is missing. See here: https://github.com/CycloneDX/cyclonedx-dotnet/pull/662/checks?check_run_id=11826932593 |
Signed-off-by: [email protected] <[email protected]>
Signed-off-by: [email protected] <[email protected]>
ab07875
to
fb008ba
Compare
Done, sorry about that |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Yeah, sorry :(
I just saw this now
Signed-off-by: [email protected] <[email protected]>
…/shawner18/cyclonedx-dotnet into unable-to-locate-bom-ref-logging
I ran into an issue where the BOM was failing to generate due to dependency resolution issues. The current logging was not clear enough to identify the source of the issue, since we had many projects and many dependencies.
Original output
Dependency ([2.29.0, )) with version range ([2.29.0, )) did not resolve to a specific version.
Unable to locate valid bom ref for my.library [2.29.0, )
Enhanced output
Dependency (my.library) with version range ([2.29.0, )) referenced by (Name:my.other.library Version:1.18.26) in did not resolve to a specific version.
Unable to locate valid bom ref for my.library [2.29.0, ) referenced in (Name: my.other.library Version: 1.18.26)