OfficeDev / TeamsFx

Developer tools for building Teams apps
Other
425 stars 163 forks source link

perf(spec-parser-tdp): list all params rather than just one #11219

Closed SLdragon closed 1 month ago

SLdragon commented 1 month ago

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

codecov[bot] commented 1 month ago

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](https://app.codecov.io/gh/OfficeDev/TeamsFx/pull/11219/graphs/tree.svg?width=650&height=150&src=pr&token=QQX8WVOEC3&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OfficeDev)](https://app.codecov.io/gh/OfficeDev/TeamsFx/pull/11219?src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OfficeDev) ```diff @@ 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](https://app.codecov.io/gh/OfficeDev/TeamsFx/pull/11219?dropdown=coverage&src=pr&el=tree&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OfficeDev) | Coverage Δ | | |---|---|---| | [packages/spec-parser/src/utils.ts](https://app.codecov.io/gh/OfficeDev/TeamsFx/pull/11219?src=pr&el=tree&filepath=packages%2Fspec-parser%2Fsrc%2Futils.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OfficeDev#diff-cGFja2FnZXMvc3BlYy1wYXJzZXIvc3JjL3V0aWxzLnRz) | `93.08% <100.00%> (-0.11%)` | :arrow_down: | | [packages/spec-parser/src/manifestUpdater.ts](https://app.codecov.io/gh/OfficeDev/TeamsFx/pull/11219?src=pr&el=tree&filepath=packages%2Fspec-parser%2Fsrc%2FmanifestUpdater.ts&utm_medium=referral&utm_source=github&utm_content=comment&utm_campaign=pr+comments&utm_term=OfficeDev#diff-cGFja2FnZXMvc3BlYy1wYXJzZXIvc3JjL21hbmlmZXN0VXBkYXRlci50cw==) | `89.14% <88.88%> (-0.37%)` | :arrow_down: |