Skip to content
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

Disable gunzip publish #474

Merged
merged 3 commits into from
Nov 28, 2024
Merged

Disable gunzip publish #474

merged 3 commits into from
Nov 28, 2024

Conversation

sateeshperi
Copy link
Contributor

No description provided.

@sateeshperi sateeshperi requested a review from a team as a code owner November 28, 2024 11:11
@sateeshperi sateeshperi changed the base branch from master to dev November 28, 2024 11:11
@nf-core nf-core deleted a comment from github-actions bot Nov 28, 2024
Copy link

github-actions bot commented Nov 28, 2024

nf-core pipelines lint overall result: Passed ✅

Posted for pipeline commit 79f1a64

+| ✅ 200 tests passed       |+
#| ❔   4 tests were ignored |#

❔ Tests ignored:

✅ Tests passed:

Run details

  • nf-core/tools version 3.0.2
  • Run at 2024-11-28 11:14:31

@sateeshperi sateeshperi self-assigned this Nov 28, 2024
@sateeshperi sateeshperi added this to the 2.8.0 milestone Nov 28, 2024
@@ -558,7 +556,7 @@
"genome_mapping_quality_histogram.png:md5,74776b1e7e8c8876af1cf89d9dbb5f31",
"genome_reads_content_per_read_position.png:md5,c6920dc357411880132c207c7d0aa68d",
"genome_uniq_read_starts_histogram.png:md5,8f7549a63eee3d3f022699aaa5bb858d",
"qualimapReport.html:md5,2c25df556c43190d52453e76de692a13",
"qualimapReport.html:md5,bbc015c0d1f3c41a08c82fb7a6839039",

Choose a reason for hiding this comment

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

Why is this changed?

Copy link
Contributor Author

Choose a reason for hiding this comment

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

@fellen31 not sure. but, it is a html file so must have changed with some formatting done recently.

This should be fine for now and will remove its md5 from snapshot if still variable in future

Copy link

@fellen31 fellen31 left a comment

Choose a reason for hiding this comment

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

LGTM!

@sateeshperi sateeshperi merged commit c9ba51a into dev Nov 28, 2024
27 checks passed
@sateeshperi sateeshperi deleted the disable-gunzip-publish branch November 28, 2024 14:14
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