Skip to content

fixed Mutect2 typo #227#236

Merged
maxulysse merged 3 commits intonf-core:devfrom
szilvajuhos:mutect2typo
Jul 15, 2020
Merged

fixed Mutect2 typo #227#236
maxulysse merged 3 commits intonf-core:devfrom
szilvajuhos:mutect2typo

Conversation

@szilvajuhos
Copy link
Copy Markdown
Contributor

@szilvajuhos szilvajuhos commented Jul 14, 2020

nf-core/sarek pull request

PR checklist

  • fixing Typo for Mutect2 #227
  • rudimentary test works
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Make sure your code lints (nf-core lint .).
  • CHANGELOG.md is updated

@szilvajuhos szilvajuhos requested a review from maxulysse as a code owner July 14, 2020 09:15
@szilvajuhos
Copy link
Copy Markdown
Contributor Author

szilvajuhos commented Jul 14, 2020

For the record:

git fetch upstream
git checkout master
git reset --hard upstream/dev
git push origin master --force 
git branch mutect2typo

The --force was missing to get rid of my obsolete commits

@maxulysse
Copy link
Copy Markdown
Member

@szilvajuhos sorry, I merged #233 so the dev version of the container no longer contain bwa but bwa-mem2, so CI is failing
Can you please update your branch?

I remember I could do that from github, but can't seem to find that anymore

@maxulysse maxulysse linked an issue Jul 14, 2020 that may be closed by this pull request
@maxulysse maxulysse added this to the 3.0 milestone Jul 14, 2020
@maxulysse maxulysse merged commit 8a31a8e into nf-core:dev Jul 15, 2020
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.

Typo for Mutect2

2 participants