-
Notifications
You must be signed in to change notification settings - Fork 63
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
Fill the ReferencedAssemblyNames Property for all loaded Assemblies #319
Conversation
d9bf4cc
to
fbdec12
Compare
Fixes issue TNG#311 Signed-off-by: Karl Rhenius <[email protected]>
fbdec12
to
7bb008e
Compare
Codecov ReportAll modified and coverable lines are covered by tests ✅
Additional details and impacted files@@ Coverage Diff @@
## main #319 +/- ##
=======================================
Coverage 65.65% 65.65%
=======================================
Files 255 255
Lines 23032 23034 +2
Branches 1962 1961 -1
=======================================
+ Hits 15121 15124 +3
+ Misses 7366 7365 -1
Partials 545 545 ☔ View full report in Codecov by Sentry. |
Hi @made-by-karl, thank you for your PR. Could you fix the code formatting? I've already created another PR that will fix the issue with the windows build |
fixes code formatting Signed-off-by: Karl Rhenius <[email protected]>
LGTM, somehow the formatting got broken again. I will reformat this and merge it after the current prerelease version has been released. |
Signed-off-by: Alexander Linne <[email protected]>
ArchBuilder always adds the names of referenced assemblies from the AssemblyDefinition.MainModule.AssemblyReferences property.
Resolves #311
Signed-off-by: Karl Rhenius [email protected]