Skip to content

Add scanoss.py as a scanner #48

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
Merged
merged 1 commit into from
Nov 25, 2021
Merged

Add scanoss.py as a scanner #48

merged 1 commit into from
Nov 25, 2021

Conversation

JustinWonjaePark
Copy link
Contributor

@JustinWonjaePark JustinWonjaePark commented Nov 25, 2021 •
edited by soimkim
Loading

Description

Add scanoss.py as a scanner.
Separate common codes for scanning.

Type of change

  • Bug fix (non-breaking change which fixes an issue)
  • New feature (non-breaking change which adds functionality)
  • Documentation update
  • Refactoring, Maintenance
  • Breaking change (fix or feature that would cause existing functionality to not work as expected)

@soimkim soimkim changed the title Seperate common codes for scanning. Add scanoss.py as a scanner Add scanoss.py as a scanner Nov 25, 2021
@JustinWonjaePark JustinWonjaePark mentioned this pull request Nov 25, 2021
Closed
5 tasks
@soimkim soimkim added the enhancement [PR/Issue] New feature or request label Nov 25, 2021
@soimkim
Copy link
Contributor

soimkim commented Nov 25, 2021

Dear @dd-jy ,
This PR is not yet merged into main. There are additional corrections to be made. (ex- comment output)
However, one more variable returned by the run_scan function has been added. If there is a call to the run_scan function, you need to fix it.

@soimkim soimkim merged commit 7856f16 into fosslight:temp2 Nov 25, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

@JustinWonjaePark JustinWonjaePark

Labels

enhancement [PR/Issue] New feature or request

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

2 participants

@JustinWonjaePark @soimkim