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

GitPod base image: Always self-update to the latest version of Nextflow. #2274

Merged
merged 2 commits into from
May 5, 2023

Conversation

ewels
Copy link
Member

@ewels ewels commented May 4, 2023

We've had some issues where the GitPod Nextflow version is behind the stable release, and doesn't include important bugfixes and changes. It's difficult to remember to bump the version there manually every time. This PR makes sure that the latest stable version is in the image automatically.

  • Don't pin the Nextflow version in the Conda install
  • Run nextflow self-update as part of the build process, in case bioconda is not up to date
  • GitPod config: Update Nextflow as an init task. Install pre-commit hooks.

@ewels
Copy link
Member Author

ewels commented May 4, 2023

NB: @mirpedrol added the Nextflow pin in #2196 with the logic that we should install the minimum supported version of Nextflow.

This PR is inverting that logic and trying to have the latest available version. I feel that this is better, as this image is mostly used for development work but also some people seem to use it for actually running pipelines as well. Having the latest version will ensure that relevant bugfixes are included, and saves us a task for remembering to keep it up to date.

@codecov
Copy link

codecov bot commented May 4, 2023

Codecov Report

Merging #2274 (5cd0b86) into dev (1fae5c2) will not change coverage.
The diff coverage is n/a.

@@           Coverage Diff           @@
##              dev    #2274   +/-   ##
=======================================
  Coverage   73.11%   73.11%           
=======================================
  Files          77       77           
  Lines        8432     8432           
=======================================
  Hits         6165     6165           
  Misses       2267     2267           

@ewels ewels merged commit 7aacb5c into nf-core:dev May 5, 2023
@ewels ewels deleted the gitpod-nextflow-latest branch May 5, 2023 06:50
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