Skip to content

feat: add transport option and golangci-lint#45

Merged
wenchy merged 4 commits intomasterfrom
add-transport-option
Jul 10, 2025
Merged

feat: add transport option and golangci-lint#45
wenchy merged 4 commits intomasterfrom
add-transport-option

Conversation

@wenchy
Copy link
Owner

@wenchy wenchy commented Jul 9, 2025

  • add golangci-lint
  • add transport option at env level and per-request level

@wenchy wenchy requested a review from Kybxd July 9, 2025 03:48
@codecov
Copy link

codecov bot commented Jul 9, 2025

Codecov Report

Attention: Patch coverage is 92.15686% with 4 lines in your changes missing coverage. Please review.

Project coverage is 80.32%. Comparing base (788748e) to head (53ee18a).
Report is 1 commits behind head on master.

Files with missing lines Patch % Lines
env.go 71.42% 1 Missing and 1 partial ⚠️
response.go 80.00% 2 Missing ⚠️
Additional details and impacted files
@@            Coverage Diff             @@
##           master      #45      +/-   ##
==========================================
+ Coverage   79.17%   80.32%   +1.14%     
==========================================
  Files           6        6              
  Lines         413      432      +19     
==========================================
+ Hits          327      347      +20     
+ Misses         62       61       -1     
  Partials       24       24              

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

@wenchy wenchy changed the title feat: add transport option at env level and per-request level feat: add transport option and golangci-lint Jul 9, 2025
@wenchy wenchy force-pushed the add-transport-option branch from 3184d89 to 4976532 Compare July 9, 2025 09:35
@wenchy wenchy force-pushed the add-transport-option branch from 4976532 to 8a0ef66 Compare July 9, 2025 09:38
@wenchy wenchy merged commit a3053fd into master Jul 10, 2025
4 checks passed
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