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

Deploy conda pkg and Docker image #125

Merged
merged 12 commits into from
Nov 22, 2023
Merged

Deploy conda pkg and Docker image #125

merged 12 commits into from
Nov 22, 2023

Conversation

pdiakumis
Copy link
Member

Some changes in the conda and Docker CI/CD. This works and the results for v0.5.0.9000 are:

Versioning

  • I added an extra dev part for versioning in bumpversion.cfg where we can use e.g. 0.5.0.9000, where the 9000 corresponds to the dev version. So if your current version is 0.5.1 and you do a bump2version --new-version 0.6.1.9000 dev, that will go to 0.6.1.9000, and you can bump that with bump2version followed by one of dev/patch/minor/major. This is close to what the tidyverse team utilise - https://r-pkgs.org/lifecycle.html#sec-lifecycle-version-number-tidyverse. We can talk about it in our next chat ;-)

GitHub Actions

  • Changed the conda-lock mechanism to export lock files for both osx-64 and linux-64. It does this in a two-step process where it first creates one large YAML file with dependencies for each platform, and then you render that into two different lock files corresponding to the platforms of choice. You can see the outputs in this PR.

Dockerfile

  • Handle the new conda-lock mechanism and use the linux-64 lock file to build the RNAsum conda env in the Docker container.

Conda

  • Pinning {gpgr}, {manhattanly}, and {RNAsum.data} to come from the umccr conda channel.

@pdiakumis pdiakumis requested a review from skanwal November 18, 2023 04:49
@pdiakumis pdiakumis merged commit 1b87550 into rnasum-rpackage Nov 22, 2023
2 checks passed
@pdiakumis pdiakumis deleted the deploy_0.5.0 branch November 22, 2023 05:32
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.

3 participants