Thanks to visit codestin.com
Credit goes to github.com

Skip to content

Conversation

@drpatelh
Copy link
Member

@drpatelh drpatelh commented Jan 24, 2020

Many thanks to contributing to nf-core/tools!

Please fill in the appropriate checklist below (delete whatever is not relevant). These are the most common things requested on pull requests (PRs).

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!
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

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

@drpatelh drpatelh mentioned this pull request Jan 24, 2020
5 tasks
@drpatelh
Copy link
Member Author

Ill make the changes you suggested in the PR to master here @maxulysse . Wrestling with the nanoseq pipeline at the moment but Ill come back to this. Thanks!

Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

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

Would be good to put a separate section at the top to explain the changes in the testing for this release..

Copy link
Member

@apeltzer apeltzer left a comment

Choose a reason for hiding this comment

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

That would be great indeed as its a bit more than usual - otherwise looking good :-)

@drpatelh
Copy link
Member Author

Added a line 👍 Let me know if its ok.

Now to figuring out why these tests are failing 😅

Copy link
Member

@ewels ewels left a comment

Choose a reason for hiding this comment

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

Couple of minor suggested tweaks, but otherwise good 👍

@drpatelh drpatelh merged commit 8fb351b into nf-core:dev Jan 27, 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.

4 participants