Skip to content

Simplify FIB atlas registration workflow - #903

Open
tieneupin wants to merge 2 commits into
mainfrom
simplify-register-atlas
Open

tieneupin wants to merge 2 commits into
mainfrom
simplify-register-atlas

Conversation

@tieneupin

Copy link
Copy Markdown
Contributor

Removes the try-except blocks in the workflow and lets the parent feedback_callback function handle any raised exceptions and database connection closures. Switches to using the register_dcg function introduced in PR #901 instead of loading the whole entry point.

@tieneupin tieneupin self-assigned this Sep 24, 2026
@tieneupin tieneupin added enhancement New feature or request server Relates to the server component fib Part of the FIB pipeline extension labels Sep 24, 2026
@tieneupin
tieneupin marked this pull request as ready for review September 24, 2026 17:58
@codecov

codecov Bot commented Sep 24, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 87.50000% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.16%. Comparing base (ae7931a) to head (f84c22a).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #903      +/-   ##
==========================================
+ Coverage   55.12%   55.16%   +0.04%     
==========================================
  Files         103      103              
  Lines       11559    11532      -27     
  Branches     1533     1533              
==========================================
- Hits         6372     6362      -10     
+ Misses       4843     4826      -17     
  Partials      344      344              
🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

This branch has not been deployed

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

Labels

enhancement New feature or request fib Part of the FIB pipeline extension server Relates to the server component

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant