-
-
Notifications
You must be signed in to change notification settings - Fork 562
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
Add parser for swift-show-dependencies.deplock
#3829
Add parser for swift-show-dependencies.deplock
#3829
Conversation
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
7fa8490
to
293eaf0
Compare
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
293eaf0
to
52efbbf
Compare
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
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.
@keshav-space LGTM! Thanks++
This works nicely for package resolution and dependency graph creation on the SCIO side. I have a couple nits for your consideration, this is ready to merge afterwards.
Also merge from latest to resolve conflicts.
One more question, I assume we cannot parse |
@AyanSinhaMahapatra We are not directly parsing the |
Signed-off-by: Keshav Priyadarshi <git@keshav.space>
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.
Thanks @keshav-space! Merging!
Reference: aboutcode-org/scancode.io#1278
Tasks
Run tests locally to check for errors.