snakemake / snakefmt

The uncompromising Snakemake code formatter
MIT License
147 stars 28 forks source link

feat: add retries to grammar #145

Closed maarten-k closed 2 years ago

maarten-k commented 2 years ago

snakemake 7.7 introduces the retries keyword. This pull request makes snakefmt also recognise this keyword.

mbhall88 commented 2 years ago

Thanks @maarten-k. Can you please also push the updated poetry.lock file? If you aren't using poetry, I can always add this in with #136

codecov[bot] commented 2 years ago

Codecov Report

Merging #145 (83ceb49) into master (d233dfa) will not change coverage. The diff coverage is n/a.

Impacted file tree graph

@@           Coverage Diff           @@
##           master     #145   +/-   ##
=======================================
  Coverage   98.20%   98.20%           
=======================================
  Files          12       12           
  Lines         949      949           
  Branches      211      211           
=======================================
  Hits          932      932           
  Misses         10       10           
  Partials        7        7           
Flag Coverage Δ
unittests 98.10% <ø> (ø)

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

Impacted Files Coverage Δ
snakefmt/parser/grammar.py 100.00% <ø> (ø)
mbhall88 commented 2 years ago

@maarten-k, if it's okay with you, I might move this over to #136 as I don't want to mess with the commits from dev I've already pulled in on that PR

maarten-k commented 2 years ago

@maarten-k, if it's okay with you, I might move this over to #136 as I don't want to mess with the commits from dev I've already pulled in on that PR

That is perfect.