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

Update triplea_maps.yaml #13035

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

VictoryFirst2
Copy link
Contributor

Update img links and description of World War I 1914 Balanced and Improved/Tournament Rules.

Update img links and description of World War I 1914 Balanced and Improved/Tournament Rules.
@beelee1
Copy link
Contributor

beelee1 commented Dec 11, 2024

Hi Victory

When you update the yaml, you need to change the version as well. It uses whole numbers, so version 2 is what you want.

I've never seen 2 maps in the same PR but I don't see anything wrong with it.

@VictoryFirst2
Copy link
Contributor Author

@beelee1 But I didn't make any changes to the map, only to the description. Do I need to change the version number for that too?

@beelee1
Copy link
Contributor

beelee1 commented Dec 13, 2024

Yea, anytime the yaml is changed the version needs to change.

Edit
The yaml version and the map version aren't usually the same. The yaml version is basically just for the yaml.

@Cernelius
Copy link
Contributor

@beelee1 But I didn't make any changes to the map, only to the description. Do I need to change the version number for that too?

Yea, anytime the yaml is changed the version needs to change.

Edit The yaml version and the map version aren't usually the same. The yaml version is basically just for the yaml.

Can you give me a link to the documentation for this process?

If he did not change anything in the map itself, I do not understand why a version upgrade is needed, because that would tell users to update the map with a new map which is the same as the one they already have.

@beelee1
Copy link
Contributor

beelee1 commented Dec 25, 2024

I don't know it for certain. It's just what I remember being told. You'd have to ask one of the devs. Last time I merged a yaml that wasn't correct, it messed up the entire lobby and Roi had to fix it, so I'm not gonna do it until someone says it's ok or they do it themselves.

Edit
The yaml PR is what tells people there is a new map version. If one doesn't want a new version, then the PR should be closed. The changes to the map have already taken place. Once the repo is updated, the next time that map is DL'd, you will get the latest version.

Edit 2
The PR should be made at this repo https://github.com/triplea-maps/world-war-i-1914-improved-main
Once it is accepted, it will automatically be updated without notifying anyone of an update.

If you want to notify people of an update, then you need to change the yaml version.

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.

3 participants