Skip to content

417, 418: modified transform to work with json from scancode and renamed configuration key for transform - #419

Merged
chinyeungli merged 8 commits into
aboutcode-org:417_json_transformfrom
srthkdb:417_json_transform
Mar 13, 2020
Merged

417, 418: modified transform to work with json from scancode and renamed configuration key for transform#419
chinyeungli merged 8 commits into
aboutcode-org:417_json_transformfrom
srthkdb:417_json_transform

Conversation

@srthkdb

@srthkdb srthkdb commented Mar 11, 2020

Copy link
Copy Markdown
Contributor
  1. Processes only the data inside "files"
  2. Automatically renames "path" to "about_resource"
  3. Written tests

@chinyeungli

Copy link
Copy Markdown
Contributor

@srthkdb Can you try to fix the failing tests?

srthkdb added 2 commits March 11, 2020 21:44
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from b5459a1 to db03ed1 Compare March 11, 2020 16:15
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from fe9d54a to ed891cf Compare March 11, 2020 17:04
@srthkdb

srthkdb commented Mar 11, 2020

Copy link
Copy Markdown
Contributor Author

@chinyeungli Fixed!

srthkdb added 2 commits March 11, 2020 22:52
…d_renamings, required_fields and field_filters respectively

Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from df270fb to 32cce1b Compare March 11, 2020 17:39
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from fde993e to b7e70a4 Compare March 11, 2020 18:23
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from fa27766 to 5007d12 Compare March 11, 2020 18:28
@srthkdb

srthkdb commented Mar 11, 2020

Copy link
Copy Markdown
Contributor Author

I have also renamed column_renamings, required_columns, and column_filters to field_renamings, required_fields, and field_filters respectively.

@srthkdb srthkdb changed the title 417: modified transform to work with json from scancode 417, 418: modified transform to work with json from scancode and renamed configuration key for transform Mar 11, 2020
Comment thread src/attributecode/transform.py Outdated
Comment thread .vscode/settings.json Outdated
Signed-off-by: Srthkdb <srthkdb@iitk.ac.in>
@srthkdb
srthkdb force-pushed the 417_json_transform branch from 06387d1 to 68b82cf Compare March 12, 2020 06:37
@chinyeungli
chinyeungli merged commit dd5eae9 into aboutcode-org:417_json_transform Mar 13, 2020
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