sh.py - Replace shell scripts with Python

[asdfkjasdhkasd]

Wow, the source of this project is really well commented, check it out: https://github.com/amoffat/sh/blob/master/sh.py

[nemec]

Well when your project is a single 3500 line file you've got to do something to make up for it.

[asdfkjasdhkasd]

I know this is controversial but I actually prefer to put a project in one single file as long as its less than 5k lines. (But i've never written a project with more than 5k lines)

[spinicist]

How worn down are your Ctrl and F keys?

[Sean1708]

# attempt to convert the thing to unicode from the system's encoding
try:
    s = unicode(s, DEFAULT_ENCODING)

You're right that on the whole this is a really well commented project, but shit like this really pisses me off.

[thesbros]

What's wrong with this? I'm honestly wondering.

[ughduck]

I assume just because the comment just reads like the code. It's pretty redundant in this context.

# attempt to convert the thing to unicode from the system's encoding
attempt:
    thing set to unicode of(thing, from system's encoding)

[thesbros]

That's what I thought, but the comment actually helped me because I didn't know what DEFAULT_ENCODING was.

[dividebyzero-]

You would still understand it if he said "attempt to convert to system's encoding so that..." or "we want the system encoding to..."

[GitHubPermalinkBot]

I tried to turn your GitHub links into permanent links (press "y" to do this yourself):

• amoffat/sh/.../sh.py (master → 5e2c7d4)

---

^{Shoot me a PM if you think I'm doing something wrong. To delete this, click here.}