Skip to content

Conversation

@Zektor7
Copy link
Contributor

@Zektor7 Zektor7 commented Dec 22, 2025

The WithQueryHook method returns a copy, the original db unchanged.

- 修改了查询钩子的配置方式,确保正确应用日志选项
- 避免了潜在的钩子未生效问题
- 更正了 WithOTEL 函数内对数据库实例的应用逻辑
- 确保查询钩子正确地附加到数据库实例上
- 避免了可能的指针操作错误,提升代码健壮性
@Zektor7 Zektor7 changed the title Fix(bunpgd)/withlog Fix(bunpgd)/WithQueryHook Dec 22, 2025
@Aoang Aoang changed the title Fix(bunpgd)/WithQueryHook fix(bunpgd): open option invalid Dec 25, 2025
@Aoang Aoang added this pull request to the merge queue Dec 25, 2025
Merged via the queue into FlowerLab:main with commit 483159e Dec 25, 2025
4 checks passed
@codecov
Copy link

codecov bot commented Dec 25, 2025

Codecov Report

❌ Patch coverage is 0% with 2 lines in your changes missing coverage. Please review.
✅ Project coverage is 55.89%. Comparing base (e70cc6a) to head (6aa92bb).
⚠️ Report is 1 commits behind head on main.

Files with missing lines Patch % Lines
bunpgd/open.go 0.00% 2 Missing ⚠️
Additional details and impacted files
@@           Coverage Diff           @@
##             main     #358   +/-   ##
=======================================
  Coverage   55.89%   55.89%           
=======================================
  Files          99       99           
  Lines        3421     3421           
=======================================
  Hits         1912     1912           
  Misses       1482     1482           
  Partials       27       27           

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

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