-
Notifications
You must be signed in to change notification settings - Fork 710
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
{tools}[foss/2024a] snakemake v8.27.0 #22141
base: develop
Are you sure you want to change the base?
{tools}[foss/2024a] snakemake v8.27.0 #22141
Conversation
Updated software
|
easybuild/easyconfigs/s/snakemake/snakemake-8.27.0-foss-2024a.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/s/snakemake/snakemake-8.27.0-foss-2024a.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/s/snakemake/snakemake-8.27.0-foss-2024a.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/s/snakemake/snakemake-8.27.0-foss-2024a.eb
Outdated
Show resolved
Hide resolved
easybuild/easyconfigs/s/snakemake/snakemake-8.27.0-foss-2024a.eb
Outdated
Show resolved
Hide resolved
Co-authored-by: Alexander Grund <[email protected]>
Co-authored-by: ocaisa <[email protected]>
98977f7
to
a7a5622
Compare
Is there a way to run the linter before commit? (EDIT: Perhaps On commit 0ccb637, the linter in the CI-tests were complaining about EDIT: Something fishy going on here. For instance, my local version of
However, that part doesn't seem to have been commit by the |
@ocaisa : Thanks for removing those blank lines at the end of those three easyconfigs. |
@boegelbot please test @ jsc-zen3 |
@ocaisa: Request for testing this PR well received on jsczen3l1.int.jsc-zen3.fz-juelich.de PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2584536429 processed Message to humans: this is just bookkeeping information for me, |
@boegelbot please test @ generoso |
@ocaisa: Request for testing this PR well received on login1 PR test command '
Test results coming soon (I hope)... - notification for comment with ID 2584558989 processed Message to humans: this is just bookkeeping information for me, |
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.
@asp8200 Thanks for having the patience to see this through. There a couple of trivial formatting/comments that have been moved/removed from previous easyconfigs (see #22141 (comment)). I can update those next week and get this merged
Test report by @boegelbot |
Test report by @boegelbot |
(created using
eb --new-pr
)