Skip to content

Configbuilder: nf-core configs create - #3949

Open
mirpedrol wants to merge 149 commits into
devfrom
configbuilder
Open

Configbuilder: nf-core configs create#3949
mirpedrol wants to merge 149 commits into
devfrom
configbuilder

Conversation

@mirpedrol

@mirpedrol mirpedrol commented Dec 1, 2025

Copy link
Copy Markdown
Member

Adding the new nf-core configs create command

jfy133 and others added 30 commits May 25, 2024 21:18
@jfy133
Add basic skeleton with welcome screen
e93b45a
@jfy133
Update changelgo
266a163
@jfy133
Fix linting failure
71a1475
@jfy133
Linting
a583018
@jfy133
Move common util functions/classes to common location
5e1dc52
@jfy133
Move textual CSS to common place
03eaa52
@jfy133
Add config type question
b82b6fb
@jfy133
Start adding basic details screen. Missing: validation. Not working: …
5f6b2a4 
…Hide URL Question of pipeline configs
@jfy133
Merge branch 'dev' into configbuilder2
e8233f5
@jfy133
Fix function calling due to move to generic location
17971e6
@jfy133
Copy over @mirpedrol 's writing functions
15f5be8
@jfy133
Start making config writing function actually write nextflow configs
4af6e99
@jfy133
Add URL saving, start adding validation: problem unless everything fi…
2d7863a 
…lled in 'NoneType + str' error
@jfy133
Merge branch 'dev' into configbuilder2
edc2d8f
@jfy133
Merge branch 'dev' into configbuilder2
e01c486
@jfy133
Small debugging, now know the issue
f0cb5e9
@jfy133
Fixing writing of parameters to the input file when no input from user
66227b7
@mirpedrol
Merge branch 'dev' of https://github.com/nf-core/tools into configbui…
f8ad151 
…lder2
@mirpedrol @jfy133
Add back configs create command
46c4c0a 
Co-authored-by: James A. Fellows Yates <jfy133@gmail.com>
@mirpedrol
update to new version of textual and rename CreateConfig classes
dc22d8f
@mirpedrol
add screen asking if the config is nf-core
544fd89
@mirpedrol
add validation of basicdetails
33e88dd
@mirpedrol
conditional validation for nf-core configs
227abc2
@mirpedrol
Merge branch 'dev' into configbuilder2
954b263
@mirpedrol
remove config author and url from pipeline configs
48ce7a5
@mirpedrol
fix custom fields using hide class and add pipeline name or path
078520a
@mirpedrol
add screen to select if config is for an HPC or not
a55d26b
@mirpedrol
Merge branch 'dev' of https://github.com/mirpedrol/tools into configb…
b143b30 
…uilder2
@mirpedrol
use 'local' for no HPC
b0ad759
@mirpedrol
add HPC configuration screen
d557970
mgeaghan added 23 commits March 30, 2026 09:33
@mgeaghan
fix test linting, fix typo in finalinfradetails
f290340
@mgeaghan
Merge remote-tracking branch 'origin/dev' into configbuilder
c092aa7
@mgeaghan
update final infra detail test snapshots
12d4da7
@mgeaghan
WIP: implementing feedback
e8f1f9d
@mgeaghan
WIP: implementing more feedback, adding cache dir memory to final inf…
56c1d81 
…ra details
@mgeaghan
WIP: implementing more feedback, finished setting default process res…
8ec71d9 
…ources
@mgeaghan
WIP: implementing more feedback
17bda0a
@mgeaghan
Fix validation for nf core pipeline configs
803b66c
@mgeaghan
WIP: continue adding tui tests
5eb11ef
@mgeaghan
finish adding tui tests, test writing configs
0420d1d
@mgeaghan
Update TUI test snapshots
d834f42
@mgeaghan
fix broken diff with dev
3879c0a
@mgeaghan
Merge remote-tracking branch 'origin/dev' into configbuilder-fix
b03285e
@mgeaghan
update manifest with new tcss path
579cf36
@mgeaghan
WIP: add unit tests for configs create
1835494
@mgeaghan
WIP: more unit tests for validation code
aa3b24f
@mgeaghan
merge latest dev changes
a083abd
@mgeaghan
finish validation unit tests
75b480c
@mgeaghan
finish unit tests, reformat, fix failing app tests
9a0c699
@mgeaghan
reformat again and fix ruff check errors
967ae26
@mgeaghan
resize infra final details screen in test
200d862
@mgeaghan
update snapshots
fb8d65a
@mgeaghan
remove empty config sections, clean up resource request formatting
3393082
@codspeed-hq

codspeed-hq Bot commented Jul 9, 2026 •
edited
Loading

Copy link
Copy Markdown
Contributor

Merging this PR will not alter performance

✅ 9 untouched benchmarks

Comparing configbuilder (c696c88) with dev (cb0ac11)1

Open in CodSpeed

Footnotes

  1. No successful run was found on dev (b03c230) during the generation of this report, so cb0ac11 was used instead as the comparison base. There might be some changes unrelated to this pull request in this report. ↩

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

configbuilder: nf-core configs create
Status: In progress

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

New functionality: nf-core config create (configbuilder)

7 participants

@mirpedrol @jfy133 @mgeaghan @fredjaya @nbtm-sh @amarinderthind @nf-core-bot