3 ms·
PR review: - Reformat with `black`; needs space around "=" - Change constants to uppercase, so YOU and FART - Probably best to use an f-string, e.g. f"{YOU}{
by jasonhansel 4y ago
PR review:
- Reformat with `black`; needs space around "="
- Change constants to uppercase, so YOU and FART
- Probably best to use an f-string, e.g. f"{YOU}{FART}"
- Needs tests
Otherwise LGTM
- quotehelp1829 4y agoYou forgot space between } and {
- jasonhansel 4y agoIt depends on whether you want the whitespace or not; if so, the ideal would be to use: messages = [YOU, FART] print(" ".join(messages) So that you can add additional messages (manually or programmatically) in the future. Always important to consider extensibility!