Thanks to visit codestin.com
Credit goes to github.com

Skip to content

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