Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use config.get to avoid KeyErrors. Don't use --release for nf-core/tools #142

Merged
merged 1 commit into from
Aug 14, 2018

Conversation

ewels
Copy link
Member

@ewels ewels commented Aug 14, 2018

No description provided.

@ewels ewels requested a review from a team August 14, 2018 17:30
@ewels ewels added the command line tools Anything to do with the cli interfaces label Aug 14, 2018
Copy link
Member

@sven1103 sven1103 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

perfekt

@codecov-io
Copy link

codecov-io commented Aug 14, 2018

Codecov Report

Merging #142 into dev will not change coverage.
The diff coverage is 100%.

Impacted file tree graph

@@          Coverage Diff          @@
##             dev    #142   +/-   ##
=====================================
  Coverage   88.9%   88.9%           
=====================================
  Files          7       7           
  Lines        730     730           
=====================================
  Hits         649     649           
  Misses        81      81
Impacted Files Coverage Δ
nf_core/lint.py 96.85% <100%> (ø) ⬆️

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 b535c7a...8e834c3. Read the comment docs.

@sven1103 sven1103 merged commit cb2bc27 into nf-core:dev Aug 14, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
command line tools Anything to do with the cli interfaces
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants