fosslight / fosslight_scanner

FOSSLight Scanner
Apache License 2.0
17 stars 17 forks source link

Setting.json with source_scanner selection #109

Closed soonhong99 closed 1 month ago

soonhong99 commented 2 months ago

Description

This PR introduces several enhancements to the FOSSLight Scanner's functionality and improvements to code quality:

Source Scanner Option Enhancements:

Code Cleanup and Optimization:

Error Fixes and Style Improvements:

These changes improve the flexibility of FOSSLight Scanner and enhance the code's readability and maintainability. In particular, the improvements to source scanner options provide users with more control, while the JSON format raw result generation feature facilitates additional analysis and integration work.

Test Results:

Reviewers are kindly requested to focus particularly on the implementation of the new source scanner options and the changes to setting.json.

Type of change

Please insert 'x' one of the type of change.

soonhong99 commented 2 months ago

setting.json 관련 scanner source -s 옵션 구현 완료 하였습니다.

확인해주세요~

soonhong99 commented 2 months ago

setting.json 관련 scanner source -j 옵션 구현 완료 하였습니다.

Can Generate raw result of scanners in json format At Your Setting.json

soimkim commented 1 month ago

PR의 Description에 수정한 사항에 대한 설명을 작성해주십시오.

soimkim commented 1 month ago

Conflict resolve해주시고, tox -e release시 에러가 없는지 확인해주십시오. (컨벤션 룰 에러도 출력됩니다.)

soonhong99 commented 1 month ago

관련 내역 수정 완료 및 description 부분에 코드 설명 넣어놨습니다.

soonhong99 commented 1 month ago

관련 내용 확인 및 수정 완료 하였습니다.

soonhong99 commented 1 month ago

FL binary --notice 옵션에서 print license 뿐만 아닌 setting.json에서 output으로 설정한 폴더명에 license가 txt로 저장되게끔 구현하였습니다.

soimkim commented 1 month ago

test_src 폴더 삭제 부탁드립니다.

soonhong99 commented 1 month ago

관련 내용 수정 완료하였습니다.

soonhong99 commented 1 month ago

관련 내용 수정 하였습니다.

soimkim commented 1 month ago

@soonhong99 , 수정 완료된 사항에는 커맨트 남겨주시고 Resolve conversation 클릭 부탁드려요. image

soonhong99 commented 1 month ago

image resolve를 눌렀는데 커멘트를 안날리면 resolve가 되지 않나요? 우선 커멘트도 날렸습니다. + unresolved 되어있던 커멘트도 resolve하였습니다.

soimkim commented 1 month ago

커맨트 안 남겨도 Resolve 누르시면 resolve처리됩니다. 😃

soimkim commented 1 month ago

fosslight_source에 대해 path_to_exclude가 적용 안 되는 파생 버그가 있음-> fosslight_source쪽 버그 인듯