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

fix: explicitly require gawk in mutect2/filter wrapper environment #498

Merged
merged 3 commits into from
Apr 10, 2024
Merged

fix: explicitly require gawk in mutect2/filter wrapper environment #498

merged 3 commits into from
Apr 10, 2024

Conversation

tedil
Copy link
Member

@tedil tedil commented Apr 9, 2024

The mutect2/filter wrapper requires the gawk version of match(…), so this PR explicitly requires gawk in the conda environment.

@coveralls
Copy link

coveralls commented Apr 9, 2024 •
edited
Loading

Coverage Status

coverage: 85.742%. remained the same
when pulling 179e39e on fix-mutect-filter-awk
into 4b89594 on main.

@tedil tedil requested a review from Nicolai-vKuegelgen April 9, 2024 13:42
Nicolai-vKuegelgen
Nicolai-vKuegelgen reviewed Apr 9, 2024
View reviewed changes
Copy link
Contributor

@Nicolai-vKuegelgen Nicolai-vKuegelgen left a comment

Choose a reason for hiding this comment

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

LGTM

@tedil tedil merged commit 6993233 into main Apr 10, 2024
6 checks passed
@tedil tedil deleted the fix-mutect-filter-awk branch April 10, 2024 12:14
@tedil tedil mentioned this pull request Jun 28, 2024
Merged
This was referenced Dec 9, 2024
Closed
Closed
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@Nicolai-vKuegelgen Nicolai-vKuegelgen Nicolai-vKuegelgen approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
3 participants
@tedil @coveralls @Nicolai-vKuegelgen