Skip to content

Flower field approaches #3935

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

Draft
wants to merge 2 commits into
base: main
Choose a base branch
from

Conversation

colinleach
Copy link
Contributor

Something for you to play with.

I got quite tangled in the wording, which is more inconsistent than I realized when we merged the exercise. The exercise name is "Flower Field", the code generally refers to "garden", and the documentation often uses "board". I think we're pretty much stuck with it (cf earlier comments on re-opening problem-specifications), but it's something to be aware of when reviewing.

I needed a reference for Walrus Operator (nobody mention PR #3585!), so went with the PEP. There may be better options?

@colinleach
Copy link
Contributor Author

I looked for Walrus operator documentation, but there's not much that's official. Maybe https://docs.python.org/3/whatsnew/3.8.html#assignment-expressions would be better for our needs? It links through to PEP 572 for anyone interested enough.

@BethanyG
Copy link
Member

Sorry - didn't see this fly past earlier. It's been a day.

I too, have been frustrated by the utter lack of thorough documentation (either from the PSF or others) around the Walrus operator. I think either of your suggestions are good. Either the PEP or the "What's new". Those are the two I see getting almost endlessly recycled in the blog posts and tutorials I've dug up. Quite unsatisfactory - but there you have it.

@BethanyG
Copy link
Member

Thank you so much for redoing this. I am so so sorry its been sitting for this long. Trying hard now to get it over the line. 😄

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.

2 participants