Skip to content

release: 2026.04.2#79

Merged
discreteds merged 6 commits into
mainfrom
release/2026.04.2
Apr 26, 2026
Merged

release: 2026.04.2#79
discreteds merged 6 commits into
mainfrom
release/2026.04.2

Conversation

@discreteds

Copy link
Copy Markdown
Member

Summary

  • Remove deprecated bridge methods from BaseDBConnection and legacy fallback branches in Ibis/Iceberg connect_default()
  • Replace redundant SecretStr isinstance guard with None check in ConnectionProfile.to_connection_string()
  • Closes final legacy-cleanup backlog items from the April 2026 settings-registry refactor

Changes since 2026.04.1

  • chore(connection): remove deprecated bridge methods and legacy fallback branches
  • chore(settings): replace SecretStr isinstance guard with None check
  • docs: add spec and plan for legacy cleanup final items

Test Plan

  • hatch run test:test-quick — 441 passed, 6 skipped

🤖 Generated with Claude Code

discreteds and others added 6 commits April 26, 2026 20:01
…ck branches

All downstream consumers now use ConnectionProfile / to_driver_kwargs().
Removes get_connection_string_template, get_connection_string_params,
get_connection_kwargs, format_connection_string from BaseDBConnection, the
isinstance(ConnectionProfile) guard and legacy path from
BaseIbisConnection.connect_default(), and the isinstance dispatch from
IcebergConnectionBase.connect_default().

Co-Authored-By: Claude Sonnet 4.6 <noreply@anthropic.com>
Upstream mountainash-settings now enforces SecretStr typing via
validate_assignment=True, making the isinstance guard redundant.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@codecov

codecov Bot commented Apr 26, 2026

Copy link
Copy Markdown

Codecov Report

❌ Patch coverage is 47.36842% with 10 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.82%. Comparing base (c95c439) to head (920eebb).
⚠️ Report is 8 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
src/mountainash_data/backends/ibis/connection.py 50.00% 5 Missing and 3 partials ⚠️
...rc/mountainash_data/backends/iceberg/connection.py 0.00% 1 Missing ⚠️
src/mountainash_data/core/settings/profile.py 0.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #79      +/-   ##
==========================================
+ Coverage   54.79%   55.82%   +1.03%     
==========================================
  Files          51       51              
  Lines        2617     2556      -61     
  Branches      288      270      -18     
==========================================
- Hits         1434     1427       -7     
+ Misses       1074     1021      -53     
+ Partials      109      108       -1     

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

@sonarqubecloud

Copy link
Copy Markdown

Quality Gate Failed Quality Gate failed

Failed conditions
E Maintainability Rating on New Code (required ≥ A)

See analysis details on SonarQube Cloud

Catch issues before they fail your Quality Gate with our IDE extension SonarQube for IDE

@discreteds discreteds merged commit ce70d46 into main Apr 26, 2026
6 of 8 checks passed
@discreteds discreteds deleted the release/2026.04.2 branch April 26, 2026 10:38
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.

1 participant