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

update samtools version + update actions + clean up #19

Merged

Conversation

katgorski
Copy link
Contributor

originally this was going to be just updating the version of samtools to 1.19 from 1.9, but i noticed a lot of extraneous files (because we didn't need transformBAM.sh in here, and it does not work directly with samtools v1.19) and also took the opportunity to update to the new github actions.

marking this as a draft to see if the actions work correctly and the image builds before making this ready.

note: the base image (rocker/tidyverse:4.2.2) does not have linux/arm64 available

@katgorski katgorski self-assigned this Feb 7, 2024
@katgorski katgorski added the enhancement New feature or request label Feb 7, 2024
did check versions of packages installed via apt, the correct versions are listed
@katgorski katgorski marked this pull request as ready for review February 7, 2024 22:06
@katgorski
Copy link
Contributor Author

it's skipping the checks on PR--not sure if that's intentional with the actions we use, it's been a while since i've worked on an image. PR is off draft now

@katgorski katgorski requested a review from a team February 7, 2024 22:08
Copy link
Member

@asboner asboner left a comment

Choose a reason for hiding this comment

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

Looks good!


- name: Set up Git
run: |
git config --global user.name "eipmgithubsvc1"
Copy link
Member

Choose a reason for hiding this comment

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

we should likely use secrets, but it's ok for now

@katgorski katgorski merged commit 9e59f48 into main Feb 8, 2024
1 check passed
@katgorski katgorski deleted the BIO-2033-update-samtools-version-in-eipm-bioinformatics branch March 5, 2024 15:41
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
enhancement New feature or request
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants