Skip to content

Add r-package containers for mhoban/assesspool#3492

Closed
mhoban wants to merge 5 commits intoBioContainers:masterfrom
mhoban:pipeline-tools
Closed

Add r-package containers for mhoban/assesspool#3492
mhoban wants to merge 5 commits intoBioContainers:masterfrom
mhoban:pipeline-tools

Conversation

@mhoban
Copy link
Copy Markdown

@mhoban mhoban commented Mar 31, 2025

Adding two containers:

conda-forge::r-poolfstat=3.0.0
r-optparse=1.7.5
r-tibble=3.2.1
r-stringr=1.5.1
r-readr=2.1.5
r-dplyr=1.1.4

and:

r-dplyr=1.1.4
r-tidyr=1.3.1
r-readr=2.1.5
r-janitor=2.2.1
r-purrr=1.0.4
r-optparse=1.7.5
r-matrixstats=1.5.0

@bgruening
Copy link
Copy Markdown
Member

Can you please not touch the other lines? I assume your editor changed many of them and remove trailing tabs.

@mhoban
Copy link
Copy Markdown
Author

mhoban commented Apr 1, 2025

Apologies! I think I synced my fork to the main repo before submitting and that was where things went wrong.

@mhoban
Copy link
Copy Markdown
Author

mhoban commented Apr 3, 2025

@bgruening is that something I should try to fix in this PR?

@bgruening
Copy link
Copy Markdown
Member

Mh you did not fix your changes. Please look add the diff, and revert all the changes to lines you have not added

@mhoban
Copy link
Copy Markdown
Author

mhoban commented Apr 3, 2025

Ok yes this is what I was asking. I’m happy to do that but it wasn’t clear to me from your first comment that this is what you were asking for.

@mhoban
Copy link
Copy Markdown
Author

mhoban commented Apr 5, 2025

I'm just going to close this and redo it. I'm not exactly sure what all went wrong.

@mhoban mhoban closed this Apr 5, 2025
@mhoban mhoban deleted the pipeline-tools branch April 5, 2025 01:20
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