Skip to content

make CLI startup faster - #4383

Merged
mashehu merged 27 commits into
nf-core:devfrom
mashehu:faster-cli
Jul 17, 2026
Merged

make CLI startup faster#4383
mashehu merged 27 commits into
nf-core:devfrom
mashehu:faster-cli

Conversation

@mashehu

@mashehu mashehu commented Jul 7, 2026

Copy link
Copy Markdown
Contributor

No description provided.

@codecov

codecov Bot commented Jul 7, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 71.11717% with 106 lines in your changes missing coverage. Please review.
✅ Project coverage is 78.18%. Comparing base (b2493f5) to head (86a5bdf).

Files with missing lines Patch % Lines
nf_core/__main__.py 29.68% 45 Missing ⚠️
nf_core/github_api.py 66.37% 38 Missing ⚠️
nf_core/pydantic_models.py 95.62% 6 Missing ⚠️
nf_core/pipelines/download/singularity.py 28.57% 5 Missing ⚠️
nf_core/version_updater.py 82.60% 4 Missing ⚠️
nf_core/commands_test_datasets.py 0.00% 3 Missing ⚠️
nf_core/synced_repo.py 57.14% 3 Missing ⚠️
nf_core/commands_pipelines.py 0.00% 1 Missing ⚠️
nf_core/pipelines/list.py 66.66% 1 Missing ⚠️

☔ View full report in Codecov by Harness.
📢 Have feedback on the report? Share it here.

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@codspeed-hq

codspeed-hq Bot commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

Merging this PR will improve performance by ×4.3

⚡ 9 improved benchmarks

Performance Changes

Mode Benchmark BASE HEAD Efficiency
WallTime test_import_main_startup 3,030.5 ms 385.8 ms ×7.9
WallTime test_cli_command_group_help_startup[test-datasets] 3,076.3 ms 452.6 ms ×6.8
WallTime test_cli_help_startup 3,083.8 ms 462.6 ms ×6.7
WallTime test_cli_command_group_help_startup[pipelines] 3,084.4 ms 465.9 ms ×6.6
WallTime test_cli_command_group_help_startup[subworkflows] 3,090.9 ms 469.1 ms ×6.6
WallTime test_cli_command_group_help_startup[modules] 3,097.9 ms 472.9 ms ×6.6
WallTime test_pipelines_schema_lint_startup 3.2 s 1 s ×3.1
WallTime test_pipelines_create_startup 3.3 s 2.6 s +27.19%
WallTime test_modules_install_startup 8.3 s 7.1 s +17.67%

Tip

Curious why this is faster? Comment @codspeedbot explain why this is faster on this PR, or directly use the CodSpeed MCP with your agent.


Comparing mashehu:faster-cli (86a5bdf) with dev (b2493f5)

Open in CodSpeed

@maxulysse maxulysse left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

I trust you

@mashehu
mashehu force-pushed the faster-cli branch 6 times, most recently from bb67738 to 0780d15 Compare July 17, 2026 09:53
@mashehu
mashehu merged commit b559ed4 into nf-core:dev Jul 17, 2026
123 checks passed
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.

2 participants