How to Delete maps From "triplea-maps" on Git

[Cernel]

@Cernel As the only conditions for reading a map as a mapskin, I think I would have it to:
start with "mapName",
immediately followed by "-",
immediately followed by any character but "-".

[LaFayette]

@Cernel said in How to Delete maps From "triplea-maps" on Git:

And how about if the map has both "-" and "skin" already in its name, like if I make a map called dragonsking-wars? Will the original map be seen as a mapskin too?

That seems to be very likely.

Anyways, I recall Veqryn didn't want to have any "-" in map's names, but using that caracter only to start the ending part of mapskins names. I

That would make sense why the 'contains dash' clause is in there. If the map contains no dashes, then the conditions givens are a lot stronger for identifying a map skin. That would almost fix the dragonsking-wars example (ie: dragonsking_wars)

########

FWIW these rules have not changed in quite a lot of time, I'm not sure if I'm a fan of them either.

[RoiEX]

@Cernel I was just about to ask the same question, I just couldn't come up with a map name containing skin.
Good example

[LaFayette]

@Cernel good suggestions for fixes; more fundamentally I think the problem is more that there is a magic naming convention at all. Instead of deriving whether something is a map skin or not, it seems more explicit if instead the map skin were to reference the map name directly. EG: we could place a file in the map skin folder "map_skin.properties" and in there, we'd have the name of the map listed, eg:

map=dragonsking-wars

If the map_skin.properties file is present, then we know we have a map-skin, otherwise it's a map.

[Cernel]

@LaFayette No: I like at least the fact that mapskins have to start with the name of the original.

[LaFayette]

@Cernel That would probably be a good convention, relying on it likely will continue to run into other magic naming problems. Since map names are relatively arbitrary, the names being associated by prefix only is a fundamental weakness of that design. This feels similar to the original problem stated in this forum thread, where naming conventions have to line up just so, it's an unnecessary source for errors.

[LaFayette]

@beelee you should be able to rename it, let me know if not: https://github.com/triplea-maps/schultz-european-war/settings

[beelee]

@LaFayette Thanks for the replies everyone. Yea what happened was is a couple years ago I added Schultz's map with the dashes then added another one with the underscores when I saw dashes weren't supposed to be used. I couldn't delete the first one and asked someone to do it but... so anyway, there are two right now.

I wasn't real familiar with git then and not a whole lot better now : ) although i did learn about the rename thing today : ) so I thought I couldn't delete the one with dashes and turns out I can't.

Yea Panther was correct as I've deleted repos before but "settings" doesn't show up for deleting the schultz map. So yea someone else will need to do it.

[LaFayette]

@beelee Ah, I suppose you need more than 'write' permissions. That repo (with dashes) is now deleted

[beelee]

@LaFayette