Skip to content

fix: update diamond access policy#1431

Merged
ZohebShaikh merged 7 commits intomainfrom
postgres-changes
Mar 11, 2026
Merged

fix: update diamond access policy#1431
ZohebShaikh merged 7 commits intomainfrom
postgres-changes

Conversation

@ZohebShaikh
Copy link
Contributor

@ZohebShaikh ZohebShaikh commented Mar 4, 2026

Postgres checks the type of tags. Postgres expects dict[str,str] I was using dict[str,int] with sql-lite

Updated both policies to adopt the type changes

authz change here

@ZohebShaikh ZohebShaikh requested a review from a team as a code owner March 4, 2026 11:25
@codecov
Copy link

codecov bot commented Mar 4, 2026

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 95.19%. Comparing base (485e0f8) to head (bc68d2a).
⚠️ Report is 1 commits behind head on main.

Additional details and impacted files
@@           Coverage Diff           @@
##             main    #1431   +/-   ##
=======================================
  Coverage   95.19%   95.19%           
=======================================
  Files          43       43           
  Lines        3119     3119           
=======================================
  Hits         2969     2969           
  Misses        150      150           

☔ 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.

@ZohebShaikh ZohebShaikh changed the title Update diamond access policy fix: update diamond access policy Mar 4, 2026
@ZohebShaikh ZohebShaikh merged commit 8e9ce20 into main Mar 11, 2026
18 checks passed
@ZohebShaikh ZohebShaikh deleted the postgres-changes branch March 11, 2026 11:56
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.

2 participants