Update sample.md

[callamd]

Pythonic import + choice method

Description

Resolves #(issue number)

What does your PR belong to?

• [ ] Website
• [x] Snippets
• [ ] General / Things regarding the repository (like CI Integration)

Types of changes

• [ ] Bug fix (non-breaking change which fixes an issue)
• [x] Enhancement (non-breaking improvement of a snippet)
• [ ] New feature (non-breaking change which adds functionality)
• [ ] Breaking change (fix or feature that would cause existing functionality to change)

Checklist:

• [ ] My code follows the code style of this project.
• [ ] My change requires a change to the documentation.
• [ ] I have updated the documentation accordingly.
• [ ] I have checked that the changes are working properly
• [ ] I have checked that there isn't any PR doing the same
• [ ] I have read the CONTRIBUTING document.

[callamd]

@Chalarangelo why do we import things without their namespace? I thought it was generally accepted that in production codebases something like namespace.method is preferable to method.

I can update this PR to add a description for random.choice; I do think it is a more pythonic way of retrieving a random element from a list.

Thanks!

[Chalarangelo]

Things are imported like this across the board for consistency and brevity. Using random.choice() seems like a good idea, you can update the description if you like.

[callamd]

If you want the code to be character by character as you've suggested you can do it.