[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

Custom content: allow hash-fenced table columns #2649

Open
wants to merge 4 commits into
base: main
Choose a base branch
from
Open

Custom content: allow hash-fenced table columns #2649

wants to merge 4 commits into from
+32 −13

Conversation

vladsavelyev
Copy link
Member
@vladsavelyev vladsavelyev commented Jun 22, 2024 •
edited
Loading

All of these TSV can be allowed as the custom content TSV, and the row starting with either FILE or #FILE should be interpreted as containing column names:

FILE	SEQUENCE	START	END	STRAND	GENE
myfile.fasta	chr1	55312	56664	+	GENE

#FILE	SEQUENCE	START	END	STRAND	GENE
myfile.fasta	chr1	55312	56664	+	GENE

#key: value
#FILE	SEQUENCE	START	END	STRAND	GENE
FILE	SEQUENCE	START	END	STRAND	GENE
myfile.fasta	chr1	55312	56664	+	GENE

#key: value
FILE	SEQUENCE	START	END	STRAND	GENE
FILE	SEQUENCE	START	END	STRAND	GENE
myfile.fasta	chr1	55312	56664	+	GENE

@codecov Codecov
Copy link
codecov bot commented Jun 22, 2024 •
edited
Loading

Codecov Report

Attention: Patch coverage is 14.28571% with 18 lines in your changes missing coverage. Please review.

Project coverage is 84.36%. Comparing base (26aafd8) to head (ae3a3b9).

Files Patch % Lines
multiqc/modules/custom_content/custom_content.py 14.28% 18 Missing ⚠️
Additional details and impacted files 
@@            Coverage Diff             @@
##             main    #2649      +/-   ##
==========================================
- Coverage   84.39%   84.36%   -0.03%     
==========================================
  Files         455      455              
  Lines       28068    28076       +8     
==========================================
  Hits        23687    23687              
- Misses       4381     4389       +8

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

@vladsavelyev vladsavelyev requested a review from ewels June 23, 2024 09:31
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@ewels ewels Awaiting requested review from ewels

At least 1 approving review is required to merge this pull request.

Assignees
No one assigned
Labels
custom-content
Projects
None yet
Milestone
MultiQC v1.23 - unit tests
Development

Successfully merging this pull request may close these issues.

None yet
1 participant
@vladsavelyev