Pseudo-PR: First code review
Many thanks to contributing to nf-core/lncpipe!
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
- [x] This comment contains a description of changes (with reason)
- [x] 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/lncpipe branch on the nf-core/test-datasets repo
- [x] Ensure the test suite passes (
nextflow run . -profile test,docker). - [x] Make sure your code lints (
nf-core lint .). - [x] Documentation in
docsis updated - [ ]
CHANGELOG.mdis updated - [x]
README.mdis updated
Learn more about contributing: https://github.com/nf-core/lncpipe/tree/master/.github/CONTRIBUTING.md
BTW, I didn't add test data here, as the test data is extremely big (about 2 Gb). I created a test data repo on testdata and stored the data at a local server that allowing remote access. Plz let me know if there is an alternative strategy.
BTW, I didn't add test data here, as the test data is extremely big (about 2 Gb). I created a test data repo on testdata and stored the data at a local server that allowing remote access. Plz let me know if there is an alternative strategy.
Hosting somewhere else is fine I guess, but please still add a separate branch for lncpipe and state that in a README in that branch so that people know where the test data resides.
Do you have an idea how long your tests are running? Or could you strip the dataset down in size while still running proper tests?
BTW, I didn't add test data here, as the test data is extremely big (about 2 Gb). I created a test data repo on testdata and stored the data at a local server that allowing remote access. Plz let me know if there is an alternative strategy.
Hosting somewhere else is fine I guess, but please still add a separate branch for lncpipe and state that in a README in that branch so that people know where the test data resides.
Do you have an idea how long your tests are running? Or could you strip the dataset down in size while still running proper tests?
Okey, i wll add a branch there. For now , the test data may take less than 10 mins with default setting . I tried stripping the dataset into chrX only for it can produce meanful result which also can promote the understanding to the result.
BTW, I didn't add test data here, as the test data is extremely big (about 2 Gb). I created a test data repo on testdata and stored the data at a local server that allowing remote access. Plz let me know if there is an alternative strategy.
Hosting somewhere else is fine I guess, but please still add a separate branch for lncpipe and state that in a README in that branch so that people know where the test data resides.
Do you have an idea how long your tests are running? Or could you strip the dataset down in size while still running proper tests?
The test dataset has been trimmed into 0.5Gb in total, and hosted on my own server. I also set the url in the .travis.yml , and make it pass the CI building. User can also down the data with wget command. I will write a detailed document it on this.