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

fix variable and correct version to parse gtf #234

Merged
merged 1 commit into from
Jun 16, 2019

Conversation

lpantano
Copy link
Contributor

Update with the right version to parse gtf and get the file tximport needs (there is a bug in there, I copy the wrong version in the last PR).

Fix variable name to match new naming format.

PR checklist

  • This comment contains a description of changes (with reason)
  • If you've fixed a bug or added code that should be tested, add tests!
  • If necessary, also make a PR on the nf-core/rnaseq branch on the nf-core/test-datasets repo
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Make sure your code lints (nf-core lint .).
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

Learn more about contributing: https://github.com/nf-core/rnaseq/tree/master/.github/CONTRIBUTING.md

@lpantano lpantano requested a review from drpatelh June 15, 2019 13:58
@drpatelh drpatelh merged commit bfac38c into salmon Jun 16, 2019
@apeltzer apeltzer deleted the lpantano-salmon-patch branch July 8, 2019 13:16
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.

2 participants