Skip to content

Conversation

@chaeh03
Copy link

@chaeh03 chaeh03 commented Sep 30, 2024

Description

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
Copy link
Contributor

soimkim commented Oct 3, 2024

@chaeh03 , 수정한 사항이 무엇인지 Description 업데이트 부탁드립니다. 😃

@soimkim soimkim added the chore [PR/Issue] Refactoring, maintenance the code label Oct 13, 2024
@soimkim
Copy link
Contributor

soimkim commented Oct 13, 2024

#191 요구사항 참고 부탁드립니다.

  1. src/fosslight_source.egg-info 폴더는 수정사항에 포함되면 안 됩니다.
  2. FOSSLight Source를 실행 후 결과가 정상 출력되는지 체크가 필요합니다. 그러나, 현재 호출하는 사항은 ScanCode 이며, ScanCode 호출 결과에 대해 확인하고 있습니다.
  3. Test 를 위한 파일은 별도로 최상단에 Python script로 추가하는 것 대신 tests 폴더에 생성합니다. 또한 pytest를 이용하도록 작성이 필요합니다.
  4. PR에 대한 action 중 Fail나는 사항이 있다면 보완이 필요합니다. (ex. build 에러난 부분 보시면 tox -e release 실행시 에러가 있음을 확인하실 수 있습니다.)
image

@soimkim soimkim marked this pull request as draft October 13, 2024 11:07
@soimkim
Copy link
Contributor

soimkim commented Nov 27, 2024

장기적으로 미진행 건으로 close합니다.

@soimkim soimkim closed this Nov 27, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

chore [PR/Issue] Refactoring, maintenance the code

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants