Skip to content
This repository has been archived by the owner on May 6, 2024. It is now read-only.

Add inv letter to rendering. #100

Merged
merged 1 commit into from
Oct 23, 2020
Merged

Add inv letter to rendering. #100

merged 1 commit into from
Oct 23, 2020

Conversation

yobibyte
Copy link
Contributor

Useful for testing when selecting items in the inventory. Have no idea how to test this programmatically, but I did test this with play.py.

@yobibyte yobibyte requested a review from heiner October 23, 2020 09:44
@facebook-github-bot facebook-github-bot added the CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed. label Oct 23, 2020
print(line.tobytes().decode("utf-8"))
except ValueError: # inv_strs not used.
print(
letter.tobytes().decode("utf-8"), line.tobytes().decode("utf-8")
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I'm almost disappointed this is not print(*(a.tobytes().decode("utf-8") for a in (letter, line))).

Almost, but not really.

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

How about print(*map(lambda x: x.tobytes().decode('utf-8'), (letter, line)))?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

It just gets worse :D

@yobibyte yobibyte merged commit a7de0d0 into master Oct 23, 2020
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CLA Signed This label is managed by the Facebook bot. Authors need to sign the CLA before a PR can be reviewed.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants