undefined | Better HN

0 pointsdrcongo3y ago0 comments

> it doesn't work with tabs

What do you mean by this? Are you indenting Python with tabs?

0 comments

Yes. The code started as being indented by tabs, so changing it now is a mess. Also, not to start a flamewar, but I've always preferred tabs to spaces in any language, I find them more reliable, easier to use...

drcongoOP3y ago

Not going to argue about your personal preference, but in Python spaces for indentation isn't just a personal preference - you'll run into a fair amount of issues with that, this one being just the tip of the iceberg.

gjulianm3y ago

Apart from isort not knowing how to deal with tabs, and changing the defaults "spaces" to "tabs" in the tools (along with line length too, because for some reason the 80 character limit is set in stone even though it's absolutely outdated) I haven't had many issues. And this has been running for years already.

dontlaugh3y ago

Not necessarily. Consistently indenting with tabs and aligning with spaces does work, but is tricky to enforce.

1 more reply

makapuf3y ago

If your import statements are indented, they must be in control statements (try/except or conditional imports, I fail to see why you would put those in a loop) thus they will be difficult to reorder if you import another module (with different name or stdlib status) on import error.

drcongoOP3y ago

I'm not sure what you mean here, but package names can be indented without conditionals or tries, as in...

  from application.module.modules import (
      Model1, Model2, Model3, Model4, Model5, Model6
  )

iSort handles this fine if you're using spaces for indentation.

eCa3y ago

> I have no idea how to <pre> on HN

  Two or more leading spaces on each line.[1]

[1] https://news.ycombinator.com/formatdoc

1 more reply

j / k navigate · click thread line to collapse

0 comments

gjulianm3y ago

drcongoOP3y ago

gjulianm3y ago

dontlaugh3y ago

Not necessarily. Consistently indenting with tabs and aligning with spaces does work, but is tricky to enforce.

1 more reply

makapuf3y ago

drcongoOP3y ago

I'm not sure what you mean here, but package names can be indented without conditionals or tries, as in...

  from application.module.modules import (
      Model1, Model2, Model3, Model4, Model5, Model6
  )

iSort handles this fine if you're using spaces for indentation.

eCa3y ago

> I have no idea how to <pre> on HN

  Two or more leading spaces on each line.[1]

[1] https://news.ycombinator.com/formatdoc

1 more reply

j / k navigate · click thread line to collapse