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

perf(spec-parser): type b oauth/api key auth support #11391

Merged
merged 4 commits into from
Apr 18, 2024
Merged

Conversation

SLdragon
Copy link
Contributor

No description provided.

Copy link

codecov bot commented Apr 17, 2024

Codecov Report

Attention: Patch coverage is 97.36842% with 1 lines in your changes are missing coverage. Please review.

Project coverage is 87.71%. Comparing base (79c26ed) to head (5918e18).

Additional details and impacted files

Impacted file tree graph

@@           Coverage Diff           @@
##              dev   #11391   +/-   ##
=======================================
  Coverage   87.71%   87.71%           
=======================================
  Files         485      485           
  Lines       29270    29283   +13     
  Branches     5685     5763   +78     
=======================================
+ Hits        25674    25686   +12     
  Misses       1774     1774           
- Partials     1822     1823    +1     
Files Coverage Δ
packages/spec-parser/src/specParser.ts 98.18% <100.00%> (-0.13%) ⬇️
packages/spec-parser/src/utils.ts 93.86% <100.00%> (+0.43%) ⬆️
packages/spec-parser/src/manifestUpdater.ts 92.61% <91.66%> (-0.12%) ⬇️

yuqizhou77
yuqizhou77 previously approved these changes Apr 17, 2024
KennethBWSong
KennethBWSong previously approved these changes Apr 17, 2024
…-udpate4172

# Conflicts:
#	packages/spec-parser/src/manifestUpdater.ts
@SLdragon SLdragon dismissed stale reviews from KennethBWSong and yuqizhou77 via 5918e18 April 17, 2024 09:29
@SLdragon SLdragon merged commit 1290774 into dev Apr 18, 2024
19 checks passed
@SLdragon SLdragon deleted the rentu-udpate4172 branch April 18, 2024 01:49
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.

3 participants