lunit-io / insight-viewer

medical image viewer component for React
https://insight-viewer.lunit.io
MIT License
34 stars 6 forks source link

VIEWER-154 / update README #402

Closed LTakhyunKim closed 1 year ago

LTakhyunKim commented 1 year ago

πŸ“ Description

README λ‚΄ 이전 버전에 ν•΄λ‹Ήν•˜λŠ” λ‚΄μš© μˆ˜μ • 및 ν•΄λ‹Ή 라이브러리의 λͺ©μ  및 μ‚¬μš©λ²•μ„ μƒμ„Έν•˜κ²Œ κΈ°μž…ν–ˆμŠ΅λ‹ˆλ‹€. μΆ”κ°€λœ λ‚΄μš©μ—μ„œ μˆ˜μ •μ΄ ν•„μš”ν•œ 사항 ν˜Ήμ€ μ’€ 더 μΆ”κ°€ν•  λ‚΄μš©μ΄ μžˆλ‹€λ©΄ μ½”λ©˜νŠΈ λΆ€νƒλ“œλ¦½λ‹ˆλ‹€.

βœ”οΈ PR Type

What kind of change does this PR introduce?

🎯 Current behavior

ꡬ 버전 git clone μ£Όμ†Œκ°€ κΈ°μž…λ˜μ–΄μžˆμŠ΅λ‹ˆλ‹€.

Issue Number: https://lunit.atlassian.net/browse/VIEWER-154

πŸš€ New behavior

ν˜„μž¬ 버전 git clone μ£Όμ†Œκ°€ κΈ°μ—…λ˜μ–΄ 있으며, μ„€μΉ˜ 방법, λΌμ΄λΈŒλŸ¬λ¦¬μ— λŒ€ν•œ μ„€λͺ…이 보닀 μƒμ„Έν•˜κ²Œ μž‘μ„±λ˜μ–΄ μžˆμŠ΅λ‹ˆλ‹€.

πŸ’£ Is this a breaking change?

LTakhyunKim commented 1 year ago

nx affected --target=test λ₯Ό μ‹€ν–‰ν•  경우 μž‘μ—… λ²”μœ„λ₯Ό μ²΄ν¬ν•˜μ—¬ μ›Œν¬ 슀페이슀 λ‚΄ μ΅œμ†Œ λ²”μœ„λ₯Ό ν…ŒμŠ€νŠΈν•˜λŠ” λ™μž‘μ΄ ν˜„μž¬ github action μ‹€νŒ¨μ˜ μ›μΈμœΌλ‘œ λ³΄μž…λ‹ˆλ‹€. README 만 μˆ˜μ •ν•˜μ—¬ λ“±λ‘λœ ν”„λ‘œμ νŠΈμ— μ „ν˜€ 영ν–₯을 주지 μ•Šμ•„ ν…ŒμŠ€νŠΈλ₯Ό ν•  ν”„λ‘œμ νŠΈκ°€ μ—†κ³ , 이둜 인해 report.json 이 export ν•˜μ§€ μ•Šμ•„, unit test μ—μ„œ μ—λŸ¬λ₯Ό λ°œμƒμ‹œν‚€λ„€μš”.

이λ₯Ό ν•΄κ²°ν•  μ‹œλ‚˜λ¦¬μ˜€λŠ” λ‹€μŒκ³Ό κ°™μŠ΅λ‹ˆλ‹€.

  1. jest coverage github action 을 μ‹€ν–‰ν•˜κΈ° μ „, npx nx affect target=test --ci --coverage --outputFile=report.json μ‹€ν–‰ν•˜μ—¬ report.json λ₯Ό μƒμ„±ν•©λ‹ˆλ‹€.

  2. jest coverage github action 을 μ‹€ν–‰ν•˜κΈ° μ „, report.json μ—¬λΆ€λ₯Ό ν™•μΈν•˜κ³ , 없을 κ²½μš°μ—” exit ν•˜μ—¬ github aciton 을 μ’…λ£Œ, μžˆμ„ κ²½μš°μ—” μœ„ github action 을 μ‹€ν–‰ν•©λ‹ˆλ‹€. μ‹€ν–‰ν•  땐, 이미 μƒμ„±λœ report.json 을 ν™œμš©ν•©λ‹ˆλ‹€. 이미 μƒμ„±λœ report.json 을 ν™œμš©ν•˜λŠ” 방법은 κΈ°μ‘΄ ν…ŒμŠ€νŠΈ λ³΄κ³ μ„œ μ‚¬μš© 의 λ‚΄μš©μ„ ν™œμš©ν•΄λ³΄λ € ν•©λ‹ˆλ‹€.

일단 이 뢀뢄은 μ’€ 더 찾아보고 적용 μ—¬λΆ€ κ²°μ •ν•΄λ³΄κ² μŠ΅λ‹ˆλ‹€. μ‹œλ‚˜λ¦¬μ˜€ 상 λ¬Έμ œκ°€ μ—†λŠ”μ§€ κ²€ν† ν•΄μ£Όμ‹œλ©΄ κ°μ‚¬ν•˜κ² μŠ΅λ‹ˆλ‹€.

deminoth commented 1 year ago

nx affected --target=test λ₯Ό μ‹€ν–‰ν•  경우 μž‘μ—… λ²”μœ„λ₯Ό μ²΄ν¬ν•˜μ—¬ μ›Œν¬ 슀페이슀 λ‚΄ μ΅œμ†Œ λ²”μœ„λ₯Ό ν…ŒμŠ€νŠΈν•˜λŠ” λ™μž‘μ΄ ν˜„μž¬ github action μ‹€νŒ¨μ˜ μ›μΈμœΌλ‘œ λ³΄μž…λ‹ˆλ‹€. README 만 μˆ˜μ •ν•˜μ—¬ λ“±λ‘λœ ν”„λ‘œμ νŠΈμ— μ „ν˜€ 영ν–₯을 주지 μ•Šμ•„ ν…ŒμŠ€νŠΈλ₯Ό ν•  ν”„λ‘œμ νŠΈκ°€ μ—†κ³ , 이둜 인해 report.json 이 export ν•˜μ§€ μ•Šμ•„, unit test μ—μ„œ μ—λŸ¬λ₯Ό λ°œμƒμ‹œν‚€λ„€μš”.

ν…ŒμŠ€νŠΈκ°€ ν•„μš”ν•œ λ³€κ²½ 사항이 μ—†λ‹€λ©΄ ν…ŒμŠ€νŠΈλ₯Ό μ§„ν–‰ν•˜μ§€ μ•ŠλŠ” 것이 맞긴 ν•©λ‹ˆλ‹€...

deminoth commented 1 year ago

jest coverage github action 을 μ‹€ν–‰ν•˜κΈ° μ „, report.json μ—¬λΆ€λ₯Ό ν™•μΈν•˜κ³ , 없을 κ²½μš°μ—” exit ν•˜μ—¬ github aciton 을 μ’…λ£Œ,

μ•žμ— job을 ν•˜λ‚˜ μΆ”κ°€ν•΄μ„œ nx print-affectedλ“±μœΌλ‘œ ν…ŒμŠ€νŠΈ μ‹€ν–‰ μ—¬λΆ€λ₯Ό λ¨Όμ € ν™•μΈν•˜λ©΄ 될 것 같은데, λ‹Ήμž₯은 이게 메인 μ΄μŠˆλŠ” μ•„λ‹ˆλ‹ˆ nx run insight-viewer:test둜 λ³€κ²½ν•΄μ„œ 일뢀 ν…ŒμŠ€νŠΈλ§Œ μˆ˜ν–‰ν•˜λ„λ‘ ν•˜κ³  PR λ§ˆλ¬΄λ¦¬ν•˜λŠ”κ²Œ λ‚˜μ„μˆ˜λ„ μžˆκ² μŠ΅λ‹ˆλ‹€.

LTakhyunKim commented 1 year ago

μ•žμ— job을 ν•˜λ‚˜ μΆ”κ°€ν•΄μ„œ nx print-affectedλ“±μœΌλ‘œ ν…ŒμŠ€νŠΈ μ‹€ν–‰ μ—¬λΆ€λ₯Ό λ¨Όμ € ν™•μΈν•˜λ©΄ 될 것 같은데, λ‹Ήμž₯은 이게 메인 μ΄μŠˆλŠ” μ•„λ‹ˆλ‹ˆ nx run insight-viewer:test둜 λ³€κ²½ν•΄μ„œ 일뢀 ν…ŒμŠ€νŠΈλ§Œ μˆ˜ν–‰ν•˜λ„λ‘ ν•˜κ³  PR λ§ˆλ¬΄λ¦¬ν•˜λŠ”κ²Œ λ‚˜μ„μˆ˜λ„ μžˆκ² μŠ΅λ‹ˆλ‹€.

λ„΅ μ•Œκ² μŠ΅λ‹ˆλ‹€ γ… γ… ... μ§€κΈˆ λ§μ”€ν•˜μ‹  것과 λΉ„μŠ·ν•˜κ²Œ ν…ŒμŠ€νŠΈ μ‹€ν–‰ ν›„ report.json 생성 μ—¬λΆ€λ₯Ό ν™•μΈν•˜κ³ , jest coverage job μ‹€ν–‰ μ‹œ, if 문으둜 μ²΄ν¬ν•˜λŠ” 방식을 μ μš©ν–ˆλŠ”λ° μ˜λ„ν•œλŒ€λ‘œ λ™μž‘ν•˜μ§„ μ•Šλ„€μš”... nx run insight-viewer:test 둜 λ³€κ²½ν•΄μ„œ 이번 PR 은 λ§ˆλ¬΄λ¦¬ν•˜κ² μŠ΅λ‹ˆλ‹€.

github-actions[bot] commented 1 year ago

Coverage report

St.:grey_question:
Category Percentage Covered / Total
🟒 Statements 92.94% 79/85
🟒 Branches 94.55% 52/55
🟑 Functions 77.78% 21/27
🟒 Lines 92.21% 71/77

Test suite run success

66 tests passing in 9 suites.

Report generated by πŸ§ͺjest coverage report action from dd69236986f6dd64721be8be66e78b618eca933e