[DO NOT MERGE] Introduce flexibility for versions in workflow-import #3154

ychiucco · 2026-01-22T16:48:12Z

Checklist before merging

I added an appropriate entry to CHANGELOG.md
I added logging to new code - if appropriate.
I merged main into the current branch.

github-actions · 2026-01-22T17:06:14Z

Coverage report

Click to see where and how coverage changed

File	Statements	Missing	Coverage	Coverage (new stmts)	Lines missing
fractal_server
exceptions.py	$This PR adds 6 to the number of statements in fractal_server/exceptions.py, taking it from 2 to 8.$	$This PR doesn't change the number of statements missing coverage in fractal_server/exceptions.py, which is 0.$	$This PR doesn't change the coverage rate in fractal_server/exceptions.py, which is 100% (8/8).$	$In this PR, 8 new statements are added to fractal_server/exceptions.py, 8 of which are covered (100%).$
main.py	$This PR adds 6 to the number of statements in fractal_server/main.py, taking it from 104 to 110.$	$This PR doesn't change the number of statements missing coverage in fractal_server/main.py, which is 1.$	$This PR adds 0.08 percentage points to the coverage rate in fractal_server/main.py, taking it from 98.33% (103/104) to 98.41% (109/110).$	$In this PR, 18 new statements are added to fractal_server/main.py, 18 of which are covered (100%).$
fractal_server/app/routes/api/v2
workflow_import.py	$This PR removes 3 from the number of statements in fractal_server/app/routes/api/v2/workflow_import.py, taking it from 92 to 89.$	$This PR adds 2 to the number of statements missing coverage in fractal_server/app/routes/api/v2/workflow_import.py, taking it from 2 to 4.$	$This PR removes 2.88 percentage points from the coverage rate in fractal_server/app/routes/api/v2/workflow_import.py, taking it from 97.27% (90/92) to 94.39% (85/89).$	$In this PR, 63 new statements are added to fractal_server/app/routes/api/v2/workflow_import.py, 59 of which are covered (93.65%).$	171-176, 188-192
Project Total

_{This report was generated by python-coverage-comment-action}

fractal_server/app/routes/api/v2/workflow_import.py

[skip ci]

fractal_server/app/routes/api/v2/workflow_import.py

introduce flexible_version

947d7b4

ychiucco linked an issue Jan 22, 2026 that may be closed by this pull request

Make workflow-import more flexible #3144

Open

This comment was marked as off-topic.

Sign in to view

unit test flexible_version

e98e2e3

ychiucco added 5 commits January 23, 2026 11:32

first version using json response

27ef7a3

using exception handler

a9fc5e8

override repr

eaf0334

add exception handler to fixture

5768a47

fix tests

46f4d3b

tcompa reviewed Jan 23, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Show resolved Hide resolved

tcompa reviewed Jan 23, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

tcompa reviewed Jan 23, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

tcompa reviewed Jan 23, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

tcompa reviewed Jan 23, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

ychiucco added 3 commits January 23, 2026 14:47

log UnreachableBranchError and raise regular exception

979d26f

new literals for TaskImportErrorData.error_reason

5ae2952

improve error_info

89efdb8

ychiucco mentioned this pull request Jan 23, 2026

Add support for workflow-import flexibility fractal-analytics-platform/fractal-web#937

Open

ychiucco added 4 commits January 23, 2026 15:31

fix test

2ea14d4

return info about all tasks

7e5b97f

change return values

191fe0a

fix tests

f8df087

tcompa changed the title ~~Introduce flexibility for versions in workflow-import~~ [DO NOT MERGE] Introduce flexibility for versions in workflow-import Jan 29, 2026

ychiucco added 3 commits February 2, 2026 12:38

Merge branch 'main' into 3144-make-workflow-import-more-flexible

3cced9f

remove taskgroup_write_access

c949b02

return empty list as error data

2d0192b

tcompa reviewed Feb 3, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

tcompa reviewed Feb 3, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

handle no version provided

063a51e

ychiucco added 8 commits February 3, 2026 16:25

fix tests

96c6e69

remove not used query param

c38bfab

remove isinstance

3950392

rename and change return type

202cf4f

docstring

9890681

[skip ci]

fix unit test

b9a3e87

remove data-tasks folder

001a95b

Merge branch 'main' into 3144-make-workflow-import-more-flexible

8bc3c51

ychiucco commented Feb 9, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

ychiucco commented Feb 9, 2026

View reviewed changes

fractal_server/app/routes/api/v2/workflow_import.py Outdated Show resolved Hide resolved

ychiucco added 4 commits February 9, 2026 10:43

remove task_id and taskgroup_id from TaskAvailable

749e680

fix test

a83e052

remove __repr__

50cf672

Merge branch 'main' into 3144-make-workflow-import-more-flexible

328e924

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

[DO NOT MERGE] Introduce flexibility for versions in workflow-import #3154

[DO NOT MERGE] Introduce flexibility for versions in workflow-import #3154

Uh oh!

ychiucco commented Jan 22, 2026

Uh oh!

This comment was marked as off-topic.

github-actions bot commented Jan 22, 2026 •

edited

Loading

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

[DO NOT MERGE] Introduce flexibility for versions in workflow-import #3154

Are you sure you want to change the base?

[DO NOT MERGE] Introduce flexibility for versions in workflow-import #3154

Uh oh!

Conversation

ychiucco commented Jan 22, 2026

Checklist before merging

Uh oh!

This comment was marked as off-topic.

github-actions bot commented Jan 22, 2026 • edited Loading Uh oh! There was an error while loading. Please reload this page.

Uh oh!

Coverage report

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Uh oh!

Reviewers

Assignees

Labels

Projects

Milestone

Development

Uh oh!

2 participants

github-actions bot commented Jan 22, 2026 •

edited

Loading