Skip to content

Commit

Permalink
Merge pull request #13378 from PennyDreadfulMTG/ignore-playtest-cards
Browse files Browse the repository at this point in the history
Skip all playtest cards in cards db, now that Scryfall have given us a way
  • Loading branch information
mergify[bot] authored Nov 29, 2024
2 parents b639559 + c9feea8 commit ac62758
Showing 1 changed file with 1 addition and 5 deletions.
6 changes: 1 addition & 5 deletions magic/multiverse.py
Original file line number Diff line number Diff line change
Expand Up @@ -22,10 +22,6 @@
KNOWN_MELDS = ['Brisela, Voice of Nightmares', 'Chittering Host', 'Hanweir, the Writhing Township',
'Urza, Planeswalker', 'Mishra, Lost to Phyrexia']

# Playtest cards that cause enough problems we'd rather skip them than include them.
BAD_PLAYTEST_CARDS = ['Bind // Liberate', 'Waste Land', 'Keeper of the Crown // Coronation of the Wilds', 'Boulder Jockey', 'Convention Maro',
'Glimpse, the Unthinkable', 'Gather, the Townsfolk', 'Rampant, Growth', 'The Colossal Dreadmaw']

def init(force: bool = False) -> bool:
return asyncio.run(init_async(force))

Expand Down Expand Up @@ -234,7 +230,7 @@ async def determine_values_async(printings: list[CardDescription], next_card_id:
if p.get('layout') not in layout.uses_canonical_namespace():
continue

if p.get('name') in BAD_PLAYTEST_CARDS:
if 'playtest' in p.get('promo_types', []):
continue

# A list of playtest cards that cause problems (to ignore). When Scryfall update bulk data to flag these we can remove this hardcoding in favor of something better.
Expand Down

0 comments on commit ac62758

Please sign in to comment.