Skip to content

Remove allgenes files from output#126

Merged
ggabernet merged 2 commits intoqbic-pipelines:devfrom
WackerO:export
May 31, 2022
Merged

Remove allgenes files from output#126
ggabernet merged 2 commits intoqbic-pipelines:devfrom
WackerO:export

Conversation

@WackerO
Copy link
Copy Markdown
Contributor

@WackerO WackerO commented May 31, 2022

Many thanks to contributing to qbic-pipelines!

This PR removes allgenes files (files containing the full gene dataset, not just DE ones) from the output. Also, the intermediate module outputs are not zipped anymore and not published.

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!
  • Ensure the test suite passes (nextflow run . -profile test,docker).
  • Documentation in docs is updated
  • CHANGELOG.md is updated
  • README.md is updated

Learn more about contributing: https://github.com/qbic-pipelines/rnadeseq/tree/master/.github/CONTRIBUTING.md

@WackerO WackerO changed the base branch from master to dev May 31, 2022 08:18
@WackerO WackerO requested a review from ggabernet May 31, 2022 08:55
Copy link
Copy Markdown
Collaborator

@ggabernet ggabernet left a comment

Choose a reason for hiding this comment

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

Looks good to me

@ggabernet ggabernet merged commit 04c80fa into qbic-pipelines:dev May 31, 2022
WackerO added a commit to WackerO/rnadeseq that referenced this pull request Jun 23, 2022
…-master warning even if the PR is from qbicpipelines/dev. Manually included the changes from qbic-pipelines#126 and 127 as those were done after 1script was created. Fixed schema. Also added myself to the authors in README.md :P
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