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

Converter from Main to Intermediate data structure #18

Merged
merged 1 commit into from
Aug 24, 2019

Conversation

abhishekspeer
Copy link
Contributor

@abhishekspeer abhishekspeer commented Aug 18, 2019

Signed-off-by: [email protected]

Transfer script added to Convert from Main to Intermediate data structure.

@abhishekspeer
Copy link
Contributor Author

Hi @swinslow. I've added the code to convert back from Main data structure to Intermediate data structure via this PR. Considering the current standard data structure we have, it's certainly not possible for us to store all the extracted data from the Intermediate structure. I've tried to include everything that what was possible. I'll try to improve this by making more changes in future. :)

@swinslow
Copy link
Member

Looks good! I won't be able to test this fully until the DCO sign-offs are fixed for #14 (so I can merge it and #19). But this is fine to merge now.

@swinslow
Copy link
Member

Actually, wait, the PR here is to spdx:master, but it should be spdx:rdfreader instead.

@abhishekspeer can you fix that, and/or submit a new PR if that's what required? I don't want to merge this into the master branch yet.

@abhishekspeer abhishekspeer changed the base branch from master to rdfreader August 24, 2019 18:41
@abhishekspeer
Copy link
Contributor Author

Hi @swinslow, apologies for that -- I've changed that base branch from master to rdfreader. You can merge it now, or let me know if there is anything else I need to change. :)

@swinslow swinslow merged commit 93e59a8 into spdx:rdfreader Aug 24, 2019
@swinslow swinslow mentioned this pull request Aug 24, 2019
@swinslow swinslow added this to the 0.1.0 milestone Jul 4, 2021
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.

2 participants