question-mark
Stuck on an issue?

Lightrun Answers was designed to reduce the constant googling that comes with debugging 3rd party libraries. It collects links to all the places you might be looking at while hunting down a tough bug.

And, if you’re still stuck at the end, we’re happy to hop on a call to see how we can help out.

Q: Can 2-line spacing around `configfile:` be switched off? Too much whitespace

See original GitHub issue

I just applied snakefmt to nextstrain/monkeypox. While I love the idea of a snakemake formatter and you’ve done a great job, the result for our main Snakefile is a bit too whitespacy.

In particular, it’s ridiculous that there is whitesapce between if not config and configfile:

And two lines between build_dir = "results" and auspice_dir = "auspice is too much for my taste.

Is there a way to configure this? I couldn’t figure out what all the config options are from the docs - all I could find were the CLI options, is there more?

from packaging import version
from augur.__version__ import __version__ as augur_version
import sys

min_version = "16.0.0"
if version.parse(augur_version) < version.parse(min_version):
    print("This pipeline needs a newer version of augur than you currently have...")
    print(f"Current augur version: {augur_version}. Minimum required: {min_version}")
    sys.exit(1)

if not config:

    configfile: "config/config_hmpxv1.yaml"


build_dir = "results"


auspice_dir = "auspice"

Issue Analytics

  • State:closed
  • Created a year ago
  • Comments:6 (3 by maintainers)

github_iconTop GitHub Comments

2reactions
bricoletccommented, Jul 11, 2022

Hey @mbhall88 i’m afraid i won’t be able to cover this before i submit my thesis, so early october. i have to invest all resources into that until then!

0reactions
mbhall88commented, Jun 29, 2022

@bricoletc do you think you’ll have a chance to look at this? I have added a failing test and opened a draft PR at #151. So maybe add on to that if you can?

It’s weird the formatter is passing build_dir and auspice_dir to black separately, but when it gets to auspice_dir it thinks it is still following the configfile keyword I think? I’m a bit lost.

Read more comments on GitHub >

github_iconTop Results From Across the Web

Remove double line spacing - Microsoft Support
Remove double line spacing ; Go to · > Line and Paragraph Spacing. · For more exact spacing, select Line Spacing Options, and...
Read more >
PyLint bad-whitespace Configuration - python - Stack Overflow
Pylint 2.6: The no-space-check option has been removed, it's no longer possible to consider empty line like a trailing-whitespace by using clever options...
Read more >
Sacha Chua's Emacs configuration
All I have to do is git pull; make , and stuff happens in an external Emacs process. Since I create Sacha.el via...
Read more >
Git - git-log Documentation
whitespace configuration. By default, trailing whitespaces (including lines that consist solely of whitespaces) and a space character that is immediately ...
Read more >
git-log - Show commit logs - Ubuntu Manpage
If full is specified, the full ref name (including prefix) will be printed. If auto is specified, then if the output is going...
Read more >

github_iconTop Related Medium Post

No results found

github_iconTop Related StackOverflow Question

No results found

github_iconTroubleshoot Live Code

Lightrun enables developers to add logs, metrics and snapshots to live code - no restarts or redeploys required.
Start Free

github_iconTop Related Reddit Thread

No results found

github_iconTop Related Hackernoon Post

No results found

github_iconTop Related Tweet

No results found

github_iconTop Related Dev.to Post

No results found

github_iconTop Related Hashnode Post

No results found