Skip to content

TAEEUN#2

Open
taenykim wants to merge 18 commits intoOOP-study-media:masterfrom
taenykim:master
Open

TAEEUN#2
taenykim wants to merge 18 commits intoOOP-study-media:masterfrom
taenykim:master

Conversation

@taenykim
Copy link
Member

@taenykim taenykim commented Apr 13, 2020

매번감사합니다~!

Copy link

@ghost ghost left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

수고하셨어요.
다른분들꺼 피드백보고 반영해볼만한게 있는지 확인해주세요

src/main.js Outdated
let tempSet = new Set()
lottos = lottos.split(',')
if (lottos.length !== 6) return false
if (bonusBall > 45 || bonusBall <= 0) return false
Copy link

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

다 상수로 바꿀 수 있지 않을까요?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant

Comments