Skip to content

Remove old configs#118

Merged
apeltzer merged 2 commits intonf-core:devfrom
apeltzer:remove_old_configs
Jan 7, 2019
Merged

Remove old configs#118
apeltzer merged 2 commits intonf-core:devfrom
apeltzer:remove_old_configs

Conversation

@apeltzer
Copy link
Copy Markdown
Member

@apeltzer apeltzer commented Jan 3, 2019

This removes all outdated former config profiles. These are now kept at https://github.com/nf-core/configs for all nf-core pipelines, and thus also for nf-core/eager!

PR checklist

  • This comment contains a description of changes (with reason)
  • 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/eager/tree/master/.github/CONTRIBUTING.md

@apeltzer
Copy link
Copy Markdown
Member Author

apeltzer commented Jan 3, 2019

@jfy133 Could you maybe add configuration profile documentation for the SHH profile at https://github.com/nf-core/configs similar to the crick documentation there?

@apeltzer
Copy link
Copy Markdown
Member Author

apeltzer commented Jan 3, 2019

We'd also have to remove the profile documentation here, as we will have that centrally coming in from the configs repository in the future :-)

@apeltzer apeltzer merged commit 8066fb7 into nf-core:dev Jan 7, 2019
@apeltzer apeltzer deleted the remove_old_configs branch January 7, 2019 19:07
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