-
Notifications
You must be signed in to change notification settings - Fork 23
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
Use logging instead of raw print #20
Open
kelesi
wants to merge
64
commits into
mtgatracker:master
Choose a base branch
from
kelesi:logging
base: master
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Conversation
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Make fork up to date
Proper arena id for Firesong and Sunspeaker
Proper arena id for Firesong and Sunspeaker
Added MANIFEST.in
add cmc property to card class
…bilities add card abilities; deprecate old cards
This manually adds the holiday promo for Llanowar Elves.
User/shawkins/rna
Add holiday promo for Llanowar Elves
…ectible add collectible, update generation script
Slightly smarter logic to find the MTGA install location.
Signed-off-by: Gene Cumm <[email protected]>
Make fork up to date
Make fork up to date
Make logging branch up to date
Make logging branch up to date
Use the 64-bit registry and environment variables rather than the 32-bit versions when looking for the Arena data files.
Adds support for the new macOS MTGA client
Arena has gone 64-bit.
Unbelievable. The LINE ENDINGS in the readme file is what has kept the readme blank this whole time. UN. BELIEVABLE. lol
The static data was very out of date and not useful any more, better to make this clear to any consumers than silently return unhelpful data.
Replace card["fieldname"] with card.get("fieldname", defaultValue) as appropriate.
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
Wrapped static and dynamic card pool generation into dedicated functions.
Raw printing replaced with logging to avoid unnecessary messages in tools dependent on the library.