Skip to content

Conversation

sunhailin-Leo
Copy link
Contributor

Describe what this PR does / why we need it

  • Add an adapter to support one of the fastest web-framework in Go. fiber

Does this pull request fix one issue?

NONE

Describe how to verify it

  • Add some example in pkg/adapter/fiber directory

Special notes for reviews

@CLAassistant
Copy link

CLAassistant commented Sep 22, 2022

CLA assistant check
All committers have signed the CLA.

@sczyh30 sczyh30 added area/integrations Issue related to integrations with open-source components to-review PRs to review labels Sep 23, 2022
@codecov-commenter
Copy link

Codecov Report

Base: 53.17% // Head: 53.17% // No change to project coverage 👍

Coverage data is based on head (e943c70) compared to base (3926bca).
Patch has no changes to coverable lines.

Additional details and impacted files
@@           Coverage Diff           @@
##           master     #486   +/-   ##
=======================================
  Coverage   53.17%   53.17%           
=======================================
  Files          91       91           
  Lines        5896     5896           
=======================================
  Hits         3135     3135           
  Misses       2414     2414           
  Partials      347      347           

Help us with your feedback. Take ten seconds to tell us how you rate us. Have a feature suggestion? Share it here.

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

@sczyh30 sczyh30 merged commit 0d804bb into alibaba:master Oct 11, 2022
@sczyh30 sczyh30 removed the to-review PRs to review label Oct 11, 2022
@sczyh30
Copy link
Member

sczyh30 commented Oct 11, 2022

Thanks for contributing and look forward to more!

binbin0325 pushed a commit to binbin0325/sentinel-golang that referenced this pull request Sep 18, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area/integrations Issue related to integrations with open-source components

Projects

None yet

Development

Successfully merging this pull request may close these issues.

5 participants