Skip to content

Add versioning and verbose option to script#11

Merged
DLBPointon merged 5 commits into
mainfrom
version
Mar 20, 2026
Merged

Add versioning and verbose option to script#11
DLBPointon merged 5 commits into
mainfrom
version

Conversation

@DLBPointon

Copy link
Copy Markdown
Collaborator

Actually add a versions output to the script, issue brought up by @prototaxites on sanger-tol/nf-core-modules#212

I think this should do it?

Still trying to use python syntax
@DLBPointon DLBPointon moved this from Todo to In progress in WRAP team board Mar 20, 2026
@prototaxites

Copy link
Copy Markdown

Can you post the output of plot_buscopainter.R -v?

@DLBPointon

DLBPointon commented Mar 20, 2026

Copy link
Copy Markdown
Collaborator Author

Can you post the output of plot_buscopainter.R -v?


(base) (busco-painter) dp24@:[Documents/busco_painter] (version):$: src/plot_buscopainter.R --version
Warning messages:
1: package ‘ggplot2’ was built under R version 4.5.2
2: package ‘tibble’ was built under R version 4.5.2
3: package ‘tidyr’ was built under R version 4.5.2
4: package ‘readr’ was built under R version 4.5.2
5: package ‘purrr’ was built under R version 4.5.2
6: package ‘dplyr’ was built under R version 4.5.2
7: package ‘stringr’ was built under R version 4.5.2
8: package ‘lubridate’ was built under R version 4.5.2
1.0.0

the package errors should dissapear in the CI, pretty sure it's just because i've had to fresh install the R packages to test.

Comment thread src/plot_buscopainter.R Outdated
suppressMessages(library(tidyverse))
suppressMessages(library(scales))

version <- "1.0.0"

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Question, should this actually be 1.0.1 now 🙋‍♂️

Copy link
Copy Markdown
Collaborator Author

Choose a reason for hiding this comment

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

...yes, yes it should. Done.

@DLBPointon
DLBPointon merged commit 427fc16 into main Mar 20, 2026
6 checks passed
@github-project-automation github-project-automation Bot moved this from In progress to Done in WRAP team board Mar 20, 2026
@DLBPointon
DLBPointon deleted the version branch March 20, 2026 16:32
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

2 participants