Skip to content
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.

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

Enable multiple input for -f and -o option #164

Merged
merged 2 commits into from
Jul 10, 2024
Merged

Conversation

JustinWonjaePark
Copy link
Contributor

@JustinWonjaePark JustinWonjaePark commented Jul 4, 2024

Description

Enable multiple input for -f option.

Example 1)
Command : $fosslight_source -o abc/output.xlsx -f csv excel
Output

  • abc/fosslight_log_src_YYMMDD_HHMM.txt
  • abc/output.csv
  • abc/output.xlsx

Example 2)
Command : $fosslight_source -o abc/output.xlsx -f excel -j
Output

  • abc/fosslight_log_src_YYMMDD_HHMM.txt
  • abc/output.xlsx
  • abc/scancode_raw_result.json
  • abc/scanner_output.wfp
  • abc/scanoss_raw_result.json

Example 3)
Command : $fosslight_source -o abc -f csv excel
Output

  • abc/fosslight_log_src_YYMMDD_HHMM.txt
  • abc/fosslight_report_src_YYMMDD_HHMM.csv
  • abc/fosslight_report_src_YYMMDD_HHMM.xlsx

Example 4)
Command : $fosslight_source -f csv excel
Output

  • fosslight_log_src_YYMMDD_HHMM.txt
  • fosslight_report_src_YYMMDD_HHMM.csv
  • fosslight_report_src_YYMMDD_HHMM.xlsx

Example 5)
Command : $fosslight_source -o abc/output.json -f csv excel
Error :

[FOSSLIGHT_SOURCE] Scan Result: Format error. The format of output file(-o:'abc/output.json') should
be in the format list(-f:'['csv', 'excel']').
Output

  • abc/fosslight_log_src_YYMMDD_HHMM.txt

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)

@JustinWonjaePark JustinWonjaePark added the enhancement [PR/Issue] New feature or request label Jul 4, 2024
@JustinWonjaePark
Copy link
Contributor Author

Build Action Failed due to the FOSSLight Util update

src/fosslight_source/cli.py Outdated Show resolved Hide resolved
@soimkim
Copy link
Contributor

soimkim commented Jul 5, 2024

@JustinWonjaePark, 실행 예시를 Description에 추가해주십시오.

src/fosslight_source/cli.py Outdated Show resolved Hide resolved
src/fosslight_source/cli.py Outdated Show resolved Hide resolved
src/fosslight_source/cli.py Outdated Show resolved Hide resolved
@JustinWonjaePark JustinWonjaePark force-pushed the dev_multi branch 2 times, most recently from cfff62d to b779667 Compare July 9, 2024 08:21
Signed-off-by: Wonjae Park <j.wonjae.park@gmail.com>
@JustinWonjaePark
Copy link
Contributor Author

@JustinWonjaePark, 실행 예시를 Description에 추가해주십시오.

추가 완료

src/fosslight_source/cli.py Outdated Show resolved Hide resolved
src/fosslight_source/cli.py Outdated Show resolved Hide resolved
Signed-off-by: Wonjae Park <j.wonjae.park@gmail.com>
@JustinWonjaePark JustinWonjaePark merged commit 3572b11 into main Jul 10, 2024
7 checks passed
@JustinWonjaePark JustinWonjaePark deleted the dev_multi branch July 10, 2024 07:07
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement [PR/Issue] New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants