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: global security #1620

Merged
merged 2 commits into from
Jul 18, 2023
Merged

feat: global security #1620

merged 2 commits into from
Jul 18, 2023

Conversation

nameoffnv
Copy link
Contributor

@nameoffnv nameoffnv commented Jul 5, 2023

Describe the PR
Add support global @Security tag. If need to disable security for an operation, an empty security tag is used.

Relation issue
#1453

Additional context

@codecov
Copy link

codecov bot commented Jul 17, 2023

Codecov Report

Patch coverage: 66.66% and project coverage change: -0.10 ⚠️

Comparison is base (c8372f6) 83.76% compared to head (2326208) 83.67%.

❗ Current head 2326208 differs from pull request most recent head f28882c. Consider uploading reports for the commit f28882c to get more accurate results

Additional details and impacted files
@@            Coverage Diff             @@
##           master    #1620      +/-   ##
==========================================
- Coverage   83.76%   83.67%   -0.10%     
==========================================
  Files          19       19              
  Lines        3752     3773      +21     
==========================================
+ Hits         3143     3157      +14     
- Misses        521      527       +6     
- Partials       88       89       +1     
Impacted Files Coverage Δ
parser.go 93.17% <61.11%> (-0.65%) ⬇️
operation.go 95.05% <100.00%> (+0.02%) ⬆️

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

@ubogdan ubogdan merged commit 1bf0078 into swaggo:master Jul 18, 2023
9 checks passed
@ubogdan
Copy link
Contributor

ubogdan commented Jul 18, 2023

@nameoffnv Thanks for your contribution!

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.

None yet

2 participants