-
Notifications
You must be signed in to change notification settings - Fork 3.2k
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
feat: Prereqs for Bioconductor script #50209
Conversation
Just realized somehow some not missing packages are getting added to "missing". I'll have to fix that |
So cool! Thanks @aliciaaevans 😎 I'm out of office now but will have a more detailed look tomorrow. |
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.
Really excellent! I verified script with and without the migration argument - both work as expected. Code is straightforward to follow, well-commented.
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.
Oh that is great! Thanks a lot @aliciaaevans
This script originally output missing CRAN packages, but I've added an option to output conda-forge packages stuck in migration. Also added an option to output a checklist in markdown format, which includes the first couple of pertinent lines from the bot output. (Inspired by #49778)
Example output from
python ./scripts/bioconductor/missingCranPackages.py --format markdown --migration_id r-base44_and_m2w64-ucrt