Skip to content

Ramp up priority of the sql_connection_string, if set.#1954

Open
niklassiemer wants to merge 5 commits intomainfrom
niklassiemer-database_connection_string
Open

Ramp up priority of the sql_connection_string, if set.#1954
niklassiemer wants to merge 5 commits intomainfrom
niklassiemer-database_connection_string

Conversation

@niklassiemer
Copy link
Copy Markdown
Member

closes #1904

Refactor sql_connection_string property to handle None case and improve error handling for invalid SQL types.
@codecov
Copy link
Copy Markdown

codecov bot commented Jan 16, 2026

Codecov Report

❌ Patch coverage is 85.18519% with 4 lines in your changes missing coverage. Please review.
✅ Project coverage is 80.37%. Comparing base (4c8fd55) to head (ed7ec61).
⚠️ Report is 13 commits behind head on main.

Files with missing lines Patch % Lines
pyiron_base/database/manager.py 60.00% 4 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main    #1954      +/-   ##
==========================================
+ Coverage   80.34%   80.37%   +0.02%     
==========================================
  Files          79       79              
  Lines       10458    10462       +4     
==========================================
+ Hits         8403     8409       +6     
+ Misses       2055     2053       -2     

☔ View full report in Codecov by Sentry.
📢 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.

@stale
Copy link
Copy Markdown

stale bot commented Feb 12, 2026

This issue has been automatically marked as stale because it has not had recent activity. It will be closed if no further activity occurs. Thank you for your contributions.

@stale stale bot added the stale label Feb 12, 2026
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Projects

None yet

Development

Successfully merging this pull request may close these issues.

PYIRONSQLCONNECTIONSTRING documentation misleading

2 participants