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

Costo por ciudadano #639

Merged
merged 1 commit into from
Oct 15, 2024
Merged

Costo por ciudadano #639

merged 1 commit into from
Oct 15, 2024

Conversation

ReyarB
Copy link
Contributor

@ReyarB ReyarB commented Oct 15, 2024

No description provided.

@ReyarB ReyarB requested a review from RecoX October 15, 2024 00:18
codescene-delta-analysis[bot]
codescene-delta-analysis bot approved these changes Oct 15, 2024
View reviewed changes
Copy link

@codescene-delta-analysis codescene-delta-analysis bot left a comment

Choose a reason for hiding this comment

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

✅ Code Health Quality Gates: OK

No scoreable code touched

View detailed results in CodeScene

RecoX
RecoX reviewed Oct 15, 2024
View reviewed changes
Codigo/Protocol.bas Show resolved Hide resolved
Codigo/Protocol.bas Show resolved Hide resolved
@Temis-gt
Copy link
Contributor

@RecoX este cambio obliga a pagar 2.500 monedas de oro SIN haber matado a nadie

@RecoX
Copy link
Member

RecoX commented Oct 15, 2024

@RecoX este cambio obliga a pagar 2.500 monedas de oro SIN haber matado a nadie

Imagino que te referis a la linea 7422 de Protocol.bas

Podes comentar abajo de mi comentario para mantener facilidad en saber de que cambio se habla.

Con respecto a la division por 2, cual es el motivo ?

@Temis-gt
Copy link
Contributor

Imagino que te referis a la linea 7422 de Protocol.bas

Podes comentar abajo de mi comentario para mantener facilidad en saber de que cambio se habla.

Con respecto a la division por 2, cual es el motivo ?

El motivo de /2 es establecer el PERDON BASE (actualmente 5000 por ciudadano matado)
Esto te lo esta cobrando SI MATAS a 1.. si no matas a nadie, el perdon te sale gratis; por lo que establecemos 5000/2 como perdón base (2500)

@RecoX RecoX merged commit 96e09b4 into master Oct 15, 2024
2 checks passed
@RecoX RecoX deleted the Perdon-cambia-minimo-2500 branch October 15, 2024 00:45
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@codescene-delta-analysis codescene-delta-analysis[bot] codescene-delta-analysis[bot] approved these changes

@RecoX RecoX RecoX approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@ReyarB @Temis-gt @RecoX