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.

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

lord_money: цикломатическая сложность #731

Closed
ssdaniel24 opened this issue Aug 25, 2022 · 0 comments
Closed

lord_money: цикломатическая сложность #731

ssdaniel24 opened this issue Aug 25, 2022 · 0 comments
Assignees
@Doloment
Labels
cyclomatic Связанное с цикломатической сложностью refactoring Касающееся крупной переработки имеющейся возможности

Comments

@ssdaniel24
Copy link
Contributor

ssdaniel24 commented Aug 25, 2022 •
edited by alek13
Loading 

Checking mods/lord/lord_money/init.lua            OK
Checking mods/lord/lord_money/money.lua           OK
Checking mods/lord/lord_money/shop.lua            1 warning

    mods/lord/lord_money/shop.lua:184:2: cyclomatic complexity of function is too high (29 > 10)

UPD:
при исправлении удалить из исключений в .github/workflows/luacheck.yml:33
@ssdaniel24 ssdaniel24 added refactoring Касающееся крупной переработки имеющейся возможности cyclomatic Связанное с цикломатической сложностью labels Aug 25, 2022
@Doloment Doloment self-assigned this Sep 30, 2024
Doloment added a commit to Doloment/lord that referenced this issue Sep 30, 2024
@Doloment
Rework lord_money.shop; fix localization; add multiexchange. Closes l…
bcc46f0 
…ord-server#731
@alek13 alek13 closed this as completed in 61eeef4 Oct 1, 2024
@github-project-automation github-project-automation bot moved this from Todo to Done in Рефакторинг Oct 1, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@Doloment Doloment

Labels
cyclomatic Связанное с цикломатической сложностью refactoring Касающееся крупной переработки имеющейся возможности
Projects
Рефакторинг
Status: Done
Milestone
No milestone
Development

No branches or pull requests
2 participants
@Doloment @ssdaniel24