Small touchup for readme

[henryiii]

Fixes #463. Does this look like a good idea to everyone?

[coveralls]

Coverage increased (+0.03%) to 82.43% when pulling c15d707 on henryiii-readme into 86363a1 on master.

[coveralls]

Coverage decreased (-0.05%) to 82.346% when pulling 00cd55b on henryiii-readme into 86363a1 on master.

[koreno]

what about the new local.cmd.ls form? I think it's a bit more pythonic than the from plumbum.cmd import ls...

[henryiii]

We should at least mention it, though importing things from packages is quite pythonic - I would say the local.cmd.ls form is handier, since you don't need multiple imports, but not more pythonic. It also downgrades more elegantly into the local[] form.

[henryiii]

How about that? Or we can change further if needed.

[koreno]

though importing things from packages is quite pythonic

Yes, when that package actually exists... a magical import is much less common than a __getattr__, hence my "less pythonic" argument.

It also downgrades more elegantly into the local[] form.
Indeed, and also to the RemoteMachine case:

from plumbum import SshMachine
remote = SshMachine(...)
remote.cmd.ls()

the import syntax doesn't help here...

[henryiii]

Okay, you've convinced me with the remote machine. I've made this a bit more obvious and reduced the magic import to a side note.

[koreno]

👍😏

…

On Fri, Sep 27, 2019, 00:01 Henry Schreiner ***@***.***> wrote: Okay, you've convinced me with the remote machine. I've made this a bit more obvious and reduced the magic import to a side note. — You are receiving this because you commented. Reply to this email directly, view it on GitHub <#469?email_source=notifications&email_token=AABMCR3DD2CHDRQYWAHT7J3QLUPKFA5CNFSM4IZLMKYKYY3PNVWWK3TUL52HS4DFVREXG43VMVBW63LNMVXHJKTDN5WW2ZLOORPWSZGOD7W6LFA#issuecomment-535684500>, or mute the thread <https://github.com/notifications/unsubscribe-auth/AABMCRZ5Q6VBWOQGNA4JSTDQLUPKFANCNFSM4IZLMKYA> .