Skip to content

chore: update deps and fix linter issues#54

Merged
zx06 merged 4 commits into
mainfrom
feature/optimize-assets
May 13, 2026
Merged

chore: update deps and fix linter issues#54
zx06 merged 4 commits into
mainfrom
feature/optimize-assets

Conversation

@zx06
Copy link
Copy Markdown
Owner

@zx06 zx06 commented May 12, 2026

Summary

  • Update all Go dependencies to latest versions
  • Fix nil context usage in TestResolveSSH_NoConfig (use context.TODO())
  • Remove unused cmd parameter from runQuery and runSchemaDump functions
  • Add XSQL_WORKDIR and XSQL_HOMEDIR environment variable support for config path overrides

Test plan

  • Run go test ./... to verify all tests pass
  • Run go mod tidy to verify no dependency issues
  • Verify the code compiles without warnings

@codecov
Copy link
Copy Markdown

codecov Bot commented May 12, 2026

Codecov Report

❌ Patch coverage is 62.50000% with 3 lines in your changes missing coverage. Please review.
✅ Project coverage is 82.83%. Comparing base (9689007) to head (dc7a7d7).
⚠️ Report is 1 commits behind head on main.
✅ All tests successful. No failed tests found.

Files with missing lines Patch % Lines
cmd/xsql/query.go 50.00% 1 Missing ⚠️
cmd/xsql/schema.go 50.00% 1 Missing ⚠️
internal/output/writer.go 50.00% 0 Missing and 1 partial ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##             main      #54      +/-   ##
==========================================
+ Coverage   81.31%   82.83%   +1.51%     
==========================================
  Files          46       46              
  Lines        3687     3687              
==========================================
+ Hits         2998     3054      +56     
+ Misses        537      475      -62     
- Partials      152      158       +6     
Flag Coverage Δ
e2e 47.02% <0.00%> (ø)
integration 47.02% <0.00%> (ø)
unittests 71.08% <62.50%> (+1.51%) ⬆️

Flags with carried forward coverage won't be shown. Click here to find out more.

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

🚀 New features to boost your workflow:
  • 📦 JS Bundle Analysis: Save yourself from yourself by tracking and limiting bundle sizes in JS merges.

@sonarqubecloud
Copy link
Copy Markdown

@zx06 zx06 merged commit dad1448 into main May 13, 2026
14 of 15 checks passed
@zx06 zx06 deleted the feature/optimize-assets branch May 13, 2026 01:46
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