Skip to content

New Reports #92

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.

Sign up for GitHub

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

Jump to bottom
Open
wants to merge 8 commits into
base: master
Choose a base branch
from
Open

New Reports #92

wants to merge 8 commits into from

Conversation

@tanaysethi
Copy link

@tanaysethi tanaysethi commented May 7, 2020

Adding 1 New Report to generate confidence score
Updating 1 report get_bom_component_origin_info.py to add Home Page URL and output in cvs format
Adding HubRestApi for a new function get_matched_components
Tanay Sethi added 4 commits May 7, 2020 17:50
Create generate_confidence_score.py
db372a8 
This is a new report to generate confidence score .
Create get_bom_component_origin_info_Updated.py
4e47a79 
ForBOM Component Info added Home Page URL and saving data in a CSV report Enhancement to get_bom_component_origin_info.py
Update HubRestApi.py
c2c9072 
Added get_matched_components
New Reports
163b1df 
Adding 1 New Report to generate confidence score Updating 1 report get_bom_component_origin_info.py to add Home Page URL and output in cvs format Adding HubRestApi for a new function get_matched_components
mkumykov
mkumykov reviewed May 7, 2020
View reviewed changes
Copy link
Contributor

@mkumykov mkumykov left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Tanay,

Your pull request shows 3266 lines changed in the HubRestApi.py
That does not sound right. Could you look into it?

Murat
@tanaysethi
Copy link
Author

tanaysethi commented May 8, 2020

Tanay,

Your pull request shows 3266 lines changed in the HubRestApi.py
That does not sound right. Could you look into it?

Murat

I have just added 1 function get_matched_components() .
Tanay Sethi added 4 commits May 14, 2020 16:45
@mkumykov
Copy link
Contributor

mkumykov commented May 23, 2020

@tanaysethi Could you replace TAB symbols with four spaces in all your files?
Otherwise it will really mess up the project.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

2 participants

@tanaysethi @mkumykov