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-tdp): list all params rather than just one #11219

Merged
merged 1 commit into from
Mar 29, 2024

Conversation

SLdragon
Copy link
Contributor

@SLdragon SLdragon commented Mar 28, 2024

Fix this: Bug 27275488: [Ask from TDP] Spec-parser list all parameters rather than just required or the first optional parameter

Copy link

codecov bot commented Mar 28, 2024

Codecov Report

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

Project coverage is 87.22%. Comparing base (09c7965) to head (36da3f8).

Additional details and impacted files

Impacted file tree graph

@@            Coverage Diff             @@
##              dev   #11219      +/-   ##
==========================================
- Coverage   87.22%   87.22%   -0.01%     
==========================================
  Files         458      458              
  Lines       27872    27871       -1     
  Branches     5604     5604              
==========================================
- Hits        24312    24310       -2     
  Misses       1788     1788              
- Partials     1772     1773       +1     
Files Coverage Δ
packages/spec-parser/src/utils.ts 93.08% <100.00%> (-0.11%) ⬇️
packages/spec-parser/src/manifestUpdater.ts 89.14% <88.88%> (-0.37%) ⬇️

@SLdragon SLdragon merged commit bad4111 into dev Mar 29, 2024
25 checks passed
@SLdragon SLdragon deleted the rentu-update3282 branch March 29, 2024 02:41
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