snakefmt icon indicating copy to clipboard operation
snakefmt copied to clipboard

feat: add support for new exclude expressions in use rule statements

Open johanneskoester opened this issue 3 years ago • 5 comments
trafficstars

Introduced by https://github.com/snakemake/snakemake/pull/1717

johanneskoester avatar Jul 19 '22 10:07 johanneskoester

Mhm, I did. Let me check.

johanneskoester avatar Jul 21 '22 08:07 johanneskoester

Oh no, I did it via the github interface, sorry.

johanneskoester avatar Jul 21 '22 08:07 johanneskoester

Mhm, I guess I somehow have destroyed the parsing for use rule statements...

johanneskoester avatar Jul 21 '22 08:07 johanneskoester

If the issue is obvious for you, could you help? If not, I will try to debug it later.

johanneskoester avatar Jul 21 '22 08:07 johanneskoester

It's not immediately obvious to me sorry. I won't be able to look at this until sometime late next week either sorry.

mbhall88 avatar Jul 21 '22 21:07 mbhall88

Codecov Report

Merging #153 (e327eb8) into master (09ed5ee) will not change coverage. The diff coverage is 100.00%.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #153   +/-   ##
=======================================
  Coverage   98.15%   98.15%           
=======================================
  Files          11       11           
  Lines         974      974           
  Branches      217      217           
=======================================
  Hits          956      956           
  Misses         10       10           
  Partials        8        8           
Flag Coverage Δ
unittests 98.15% <100.00%> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

Impacted Files Coverage Δ
snakefmt/parser/syntax.py 99.05% <100.00%> (ø)

codecov[bot] avatar Nov 08 '22 05:11 codecov[bot]