Skip to content

Conversation

@agathazeren
Copy link

@agathazeren agathazeren commented Mar 8, 2024

It's left as a draft because i need to eliminate the commenting out of the future imports. (probably either setting up virtualenv and using an older version of python or updating the codebase, but maybe it's something else that I have to research.) It also should be updated to use: #2912 once that lands

Sample run:

~/code/buttonmen/tools/api-client/python> ./cartesian "Cowboy Bebop"
Ed
---
Faye 3
Jet 1

Ein
---
Faye 3
Jet 2
Spike 2

Faye
---
Spike 3
Vicious 3

Jet
---
Spike 4

Spike
---

Vicious
---

@blackshadowshade
Copy link
Contributor

I reckon this could do with a little bit of commenting so that it's obvious what it does---both for others and for future you. :)

@agathazeren
Copy link
Author

I agree. I'm not sure when I will get this sorted, but I'll try to get to it in not too long.

agathazeren and others added 2 commits April 1, 2024 13:30
Counts the total number of games required to meet the limit value for all
included buttons.
@blackshadowshade
Copy link
Contributor

Just noting that the correct way to be bringing in updates into your branch is to rebase your branch instead of merging our changes. This means that your branch becomes relative to our current code status and that we don't have merge issues.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants