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

Change the json key and value format #99

Merged
merged 1 commit into from
Jul 8, 2024
Merged

Conversation

soimkim
Copy link
Contributor

@soimkim soimkim commented Jul 8, 2024

Description

Change the json key and value format.

Type of change

Please insert 'x' one of the 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 self-assigned this Jul 8, 2024
@soimkim soimkim added the chore [PR/Issue] Refactoring, maintenance the code label Jul 8, 2024
@soimkim
Copy link
Contributor Author

soimkim commented Jul 8, 2024

@SeongjunJo , 실행 parameter에 맞게 setting.json의 key값과 value 포맷을 변경했습니다.

  • "type": "analyze" : 읽지 않는 값 삭제
  • link : 배열 아닌 string으로 받음
  • output 하나로 받음. (dir, file 분리하여 받지 않음)
  • exclude_path 대신 key값 exclude로 변경

@soimkim soimkim force-pushed the apply_setting_from_json branch from 588420c to 44c6882 Compare July 8, 2024 06:31
@soimkim soimkim merged commit e889cf9 into main Jul 8, 2024
6 checks passed
@soimkim soimkim deleted the apply_setting_from_json branch July 8, 2024 07:05
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.

1 participant