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.

Sign up for GitHub

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

Jump to bottom

fix: name of ability attribute #133

Merged

oleneyl merged 1 commit into simaple-team:main from jj9810:ability-fetch-minorfix

Jan 11, 2023

Collaborator

jj9810 commented Jan 11, 2023 •

edited

Loading

제가 #71 에서 추가한 코드들 중 임의로 붙인 이름들을 모듈 내에서 사용되는 이름으로 통일시키는 PR입니다.

추가로 이하의 속성들도 제가 임의로 카멜케이스로 붙였는데, 이것들에 대한 정식 명칭을 정할 수 있으면 좋을 것 같습니다.

strToDEX: 직접투자 STR의 n% -> DEX
dexToSTR: 직접투자 DEX의 n% -> STR
intToLUK: 직접투자 INT의 n% -> LUK
lukToDEX: 직접투자 LUK의 n% -> DEX
relativeAttackPower: n레벨마다 공격력 1 증가
relativeMagicAttack: n레벨마다 공격력 1 증가
passiveSkill: 패시브 스킬 레벨 n 증가
multiTarget: 다수 공격 스킬의 공격 대상 n 증가


          fix: name of ability attribute

8d589d8

oleneyl approved these changes

View reviewed changes

Collaborator

oleneyl commented Jan 11, 2023

다른 명칭 명명하는건 이슈로 남겨주세요.

oleneyl merged commit 40a1a0a into simaple-team:main

jj9810 deleted the ability-fetch-minorfix branch

January 11, 2023 16:55

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

Labels

None yet