search

kyleam / snakemake-mode

[MIRROR] Emacs support for Snakemake
https://git.kyleam.com/snakemake-mode/about/
GNU General Public License v3.0
33 stars 8 forks source link

include statements not indented correctly? #16

Closed endrebak closed 8 years ago

endrebak commented 8 years ago

When I write

include:
include_prefix + "/rules/window_counts/csaw_window_counts.rules"

the region is not indented properly, not even when I mark the region and try to indent it. Glitch? Was hoping for the following, like in the official workflow repo:

include:
    include_prefix + "/bio/ngs/rules/mapping/peanut.rules"
kyleam commented 8 years ago

Glitch?

Yes, thanks. I'll add this to the list of things to address when I rework the indentation functions.

kyleam commented 8 years ago

when I rework the indentation functions

... which I'm doing as part of the changes mentioned here.

endrebak commented 8 years ago

(Just adding other indentation quirks in this thread)

When there is an empty line between e.g. input: and the next, the line is not properly indented.

input:

expand(...

This is a borderline case, I know.

edit: Come to think of it, the above might be important to fix, since it happens in the run: portion of a rule too, where you might want an empty line for aesthetic reasons.

run:

    warnings.filterwarnings("ignore", category=UserWarning)
kyleam commented 8 years ago

These cases should be fixed by 1e404ec. Let me know if you run into any issues with the new indentation implementation.