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.

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

System18: Remove discarded trains from the game #11275

Open
wants to merge 2 commits into
base: master
Choose a base branch
from

Conversation

vandamm
Copy link
Collaborator

@vandamm vandamm commented Oct 12, 2024

Fixes #11264

Before clicking "Create"

  • Branch is derived from the latest master
  • Add the pins or archive_alpha_games label if this change will break existing games
  • Code passes linter with docker compose exec rack rubocop -a
  • Tests pass cleanly with docker compose exec rack rake

May require pins if in any game discarded train was bought

@vandamm vandamm requested a review from roseundy October 12, 2024 21:03
@vandamm vandamm added System18 pins PR that will require some games to be pinned labels Oct 12, 2024
@roseundy
Copy link
Collaborator

Sorry, this is not the correct way to fix this.

Take a look at the variable DISCARDED_TRAINS

Copy link
Collaborator

@roseundy roseundy left a comment

Choose a reason for hiding this comment

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

Please look at the variable defined in the Game base class: DISCARDED_TRAINS
This just needs to be changed in the map method map_*_constants

@vandamm
Copy link
Collaborator Author

vandamm commented Oct 13, 2024

@roseundy sorry did not find this and found 1822 implementation instead.
Fixed to using DISCARDED_TRAINS = :remove in the base System18 because it's a base rule. Checked all implemented maps and none of them need to override it per the rules

@vandamm vandamm requested a review from roseundy October 13, 2024 11:42
@roseundy roseundy added archive_alpha_games Needs alpha games archiving and removed pins PR that will require some games to be pinned labels Oct 19, 2024
@roseundy
Copy link
Collaborator

We are still archiving System18 games, not pinning them.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
archive_alpha_games Needs alpha games archiving System18
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[System18: Poland] Discarded train available for purchase
2 participants