Skip to content

Conversation

@tahitihat
Copy link

This implements everything in the accounts example specification.

elif args.subcommand == 'list':
L1 = ("Date Amount Balance Memo\n"
"---------- ------- ------- ----")
print(L1)
Copy link
Owner

Choose a reason for hiding this comment

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

There's nothing wrong with what you have here. However, it is a general principle in python that in most places where you could use a variable name you can instead use an expression. This applies particularly to function arguments. You have "L1 = " followed by "print(L1)". You could instead write "print()". That is, the above two lines would become:

print("Date         Amount  Balance  Memo\n"
         "----------  -------  -------  ----")

This is how you will typically see print statements written in Python programs.

@bitdancer
Copy link
Owner

Added several review comments. If you don't get to them before, addressing these would be the perfect thing for those extra hours in your first week, if you in fact have any :)

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