Based on a thorough review of attribute data in cards.json, and comparison to the actual cards, this PR contains the following changes:
Updates supply attribute for Hermit (Dark Ages) to true, so it appears in sets
Updates supply attribute for Spoils (Dark Ages) to false, so it does not erroneously appear in sets
Updates villagers to false for a great many cards in Renaissance and Menagerie, so the Gameplay Setup no longer shows unnnecessary Villagers Tokens when those cards are selected
A large number of data fixes for individual cards with missing or incorrect values for trash, cards, coins, etc.
Note that many of these are not exact representations of the data on the card (as many cards grant a variable number of a given attribute), but I adhered to established conventions as closely as possible (a card whose primary action could grant 0 or more cards is configured as "cards": 1, cards with "If it is an... Action card, +1 Action. Treasure card, +1 Coin. Victory card, +1 Card" get all three of those attributes set to 1, etc.).
Please let me know if you have questions/comments/suggestions/etc.!
Based on a thorough review of attribute data in cards.json, and comparison to the actual cards, this PR contains the following changes:
supplyattribute for Hermit (Dark Ages) totrue, so it appears in setssupplyattribute for Spoils (Dark Ages) tofalse, so it does not erroneously appear in setsvillagerstofalsefor a great many cards in Renaissance and Menagerie, so the Gameplay Setup no longer shows unnnecessary Villagers Tokens when those cards are selectedtrash,cards,coins, etc."cards": 1, cards with "If it is an... Action card, +1 Action. Treasure card, +1 Coin. Victory card, +1 Card" get all three of those attributes set to1, etc.).Please let me know if you have questions/comments/suggestions/etc.!