-
Notifications
You must be signed in to change notification settings - Fork 192
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
modify template to use nf-validation plugin #1771
Closed
Closed
Changes from 14 commits
Commits
Show all changes
66 commits
Select commit
Hold shift + click to select a range
c623783
update branch
mirpedrol c29f9eb
fix linting
mirpedrol 566bb58
remove check_samplesheet.py
mirpedrol 26db854
merge branch dev into schema-validation
mirpedrol 0ccb9bc
remove NFcoreSchema.groovy from pipeline linting
mirpedrol 8b4d0a5
update validation params
mirpedrol 8c66213
update plugin version
mirpedrol bc75e05
use fromSamplesheet()
mirpedrol cd9dadf
refactoring
mirpedrol fe6da9b
refactoring according to nf-validation params
mirpedrol 6dec44c
update nextflow_schema.json
mirpedrol 39ad411
fix padding
mirpedrol 9b08dd9
add link to nf-validation documentation
mirpedrol 8fc5840
Merge branch 'dev' into schema-validation
mirpedrol fd5f963
refactor schema_ignore_params to validationSchemaIgnoreParams
mirpedrol 15e3ec2
Merge branch 'schema-validation' of https://github.com/mirpedrol/tool…
mirpedrol 0658d0b
Add -t / --tower option to 'nf-core download'.
MatthiasZepper 4d7cfd9
Intermediate commit
MatthiasZepper bc77266
Implement logic for the Tower download in DownloadWorkflow:download_w…
MatthiasZepper 66cb1a0
Extend ModulesRepo:setup_local_repo() with a cache_only bool, so we c…
MatthiasZepper 60d8309
Create WorkflowRepo subclass of ModuleRepo and initialise local clone.
MatthiasZepper a431dfd
TypeError: HEAD is a detached symbolic reference as it points to ...
MatthiasZepper 3196e41
Split history ./modules/modules_repo.py to synced_repo.py
MatthiasZepper d0dea84
Split history ./modules/modules_repo.py to synced_repo.py
MatthiasZepper 4fcdcfd
Split history ./modules/modules_repo.py to synced_repo.py
MatthiasZepper fb6a6e6
Duplication of ModulesRepo to SyncedRepo done.
MatthiasZepper a3ccffa
Strip ModulesRepo class of the methods moved to new superclass.
MatthiasZepper 1f0e08a
Rebase to current dev.
MatthiasZepper 62d25d0
Local caching of the repo works now.
MatthiasZepper 621a8e1
Started implementing the config download.
MatthiasZepper c478d9a
Started to implement the multiple revision selection for the Tower do…
MatthiasZepper 67664da
Rewrite get_revision_hash() function to accomodate multiple revisions.
MatthiasZepper c9cd858
The 2nd revivial of the config choice. Now available for archives wit…
MatthiasZepper 2cfd159
Inclusion of the revision in the output file name is problematic with…
MatthiasZepper 3b7019f
Allow multiple instances of the -r argument. Needed for scripted down…
MatthiasZepper a92c3f4
Finished updating the prompts for the dialogues.
MatthiasZepper cee66b5
Converted the self.wf_download_url into a dict.
MatthiasZepper 8708d31
Enable multi-revision classic download.
MatthiasZepper e545df5
Small tweaks to ensure that tools doesn't bail out if there is no sym…
MatthiasZepper 23cfbf6
Initialise the Git repo clone of the workflow.
MatthiasZepper d94354f
WorkflowRepo attributes and functions.
MatthiasZepper 7dbcca4
Finished the Tower download branch.
MatthiasZepper faac11b
Minor tweaks to the container download functionality.
MatthiasZepper 9ca946d
Updating docs and changelog, fixing linting errors.
MatthiasZepper 7204862
Hopefully fixed the existing tests. New ones still need to be written.
MatthiasZepper 3a60a54
Refactor the CLI commands for the Singularity Cache Dir
MatthiasZepper b53c9fd
Readme updates for the new remote Singularity cache feature.
MatthiasZepper 2c6764d
Add interactive check in retry for parsing the index.
MatthiasZepper 2e8d0d2
Incorporating some suggestions by @mashehu.
MatthiasZepper c86df5b
Apply suggestions from code review @mashehu
MatthiasZepper e21f2a2
Writing additional tests for the --tower download functionality.
MatthiasZepper 3c62b4d
Move alterations from Version 2.8 (which this PR didn't make anymore)…
MatthiasZepper 3e0e24a
Adding the info about remote containers to the summary log rather tha…
MatthiasZepper ec2b8c0
Moved the notification about remote containers to summary_log.
MatthiasZepper 6d82c53
Apply suggestions from code review
MatthiasZepper b52e8e1
Fixes suggested by @mirpedrol during review. Thanks!
MatthiasZepper 02c1060
@mashehu suggested that downloading the containers should not be opti…
MatthiasZepper 178121b
Bugfix: WorkflowRepo.tidy_tags() did indeed only tidy tags. However, …
MatthiasZepper 141ab7d
rm dockerfile + add citations
SusiJo 6dcf961
rm outdated Dockerfile
SusiJo 2590eb5
add version and subpath to multiqc report comment link
mashehu 1340d42
update changelog
mashehu 5473fd6
also add version to main link
mashehu b2c92f1
fix linting test
mashehu db4774a
update multiqc_config during version bump
mashehu d53a975
bump version in multiqc_config correctly
mashehu File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This file was deleted.
Oops, something went wrong.
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.