Skip to content

Catch AssertionError from schema build#1267

Merged
ewels merged 4 commits into
devfrom
ewels-patch-1
Nov 16, 2021
Merged

Catch AssertionError from schema build#1267
ewels merged 4 commits into
devfrom
ewels-patch-1

Conversation

@ewels

@ewels ewels commented Sep 7, 2021

Copy link
Copy Markdown
Member

Fixes bug found by @erikrikarddaniel where error from nextflow config raised an AssertionError that was not handled.

Untested.

Fixes bug found by @erikrikarddaniel where error from `nextflow config` raised an `AssertionError` that was not handled.
@github-actions

This comment has been minimized.

@ewels

ewels commented Sep 7, 2021

Copy link
Copy Markdown
Member Author

To replicate error, introduce a syntax error in nextflow.config etc. Without this fix you should get a big traceback. With the fix you should get a nicely formatted message that prints the nextflow error message.

@ewels

ewels commented Sep 7, 2021

Copy link
Copy Markdown
Member Author

Something fishy going wrong with the Black GitHub action - seems to be an installation error with Black, nothing related to the code change.

@ewels

ewels commented Sep 7, 2021

Copy link
Copy Markdown
Member Author

Hopefully this PR should fix the actions error: #1268

@codecov

codecov Bot commented Nov 16, 2021

Copy link
Copy Markdown

Codecov Report

Merging #1267 (a135b8e) into dev (f58077e) will increase coverage by 0.01%.
The diff coverage is 0.00%.

Impacted file tree graph

@@            Coverage Diff             @@
##              dev    #1267      +/-   ##
==========================================
+ Coverage   67.52%   67.54%   +0.01%     
==========================================
  Files          50       50              
  Lines        5611     5611              
==========================================
+ Hits         3789     3790       +1     
+ Misses       1822     1821       -1     
Impacted Files Coverage Δ
nf_core/__main__.py 55.60% <0.00%> (ø)
nf_core/utils.py 84.50% <ø> (+0.25%) ⬆️

Continue to review full report at Codecov.

Legend - Click here to learn more
Δ = absolute <relative> (impact), ø = not affected, ? = missing data
Powered by Codecov. Last update ffab773...a135b8e. Read the comment docs.

@ewels

ewels commented Nov 16, 2021

Copy link
Copy Markdown
Member Author

Tested locally and seems to be working well ✅

@ewels ewels enabled auto-merge November 16, 2021 23:51
@ewels ewels disabled auto-merge November 16, 2021 23:54
@ewels ewels merged commit bc8e01c into dev Nov 16, 2021
@ewels ewels deleted the ewels-patch-1 branch November 16, 2021 23:54
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant