Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

feat: Add Host method #114

Merged
merged 1 commit into from
May 2, 2023
Merged

feat: Add Host method #114

merged 1 commit into from
May 2, 2023

Conversation

devhaozi
Copy link
Member

@devhaozi devhaozi commented May 2, 2023

@devhaozi devhaozi self-assigned this May 2, 2023
@devhaozi devhaozi requested a review from hwbrzzl May 2, 2023 12:40
@devhaozi devhaozi enabled auto-merge (squash) May 2, 2023 12:41
@codecov
Copy link

codecov bot commented May 2, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.02 ⚠️

Comparison is base (c7df6df) 53.32% compared to head (b8333d5) 53.30%.

Additional details and impacted files
@@            Coverage Diff             @@
##           master     #114      +/-   ##
==========================================
- Coverage   53.32%   53.30%   -0.02%     
==========================================
  Files         105      105              
  Lines        7483     7485       +2     
==========================================
  Hits         3990     3990              
- Misses       3214     3216       +2     
  Partials      279      279              
Impacted Files Coverage Δ
http/gin_request.go 0.00% <0.00%> (ø)

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

Copy link
Contributor

@hwbrzzl hwbrzzl left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM

@devhaozi devhaozi merged commit 547bf96 into master May 2, 2023
@devhaozi devhaozi deleted the haozi/add-host-method branch May 2, 2023 13:46
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

✨ [Feature] Add ctx.Request().Host() method
2 participants