-
Notifications
You must be signed in to change notification settings - Fork 188
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 to separate package #10842
Conversation
Co-authored-by: turenlong <rentu@microsoft.com>
* perf: add test files * perf: update test
* test: add browser test * perf: remove useless code
Co-authored-by: turenlong <rentu@microsoft.com>
* ci: add ci for spec-parser * test: update test * test: update test * test: update dependency
* ci: add cd for spec parser * fix: fix pacakge * test: fix ut
* perf: try to fix browser lib issue * perf: add missing constant export --------- Co-authored-by: turenlong <rentu@microsoft.com>
* perf: add format for spec parser * fix: add eslint file * build: update lock file
* perf: add license and update package.json * ci: update version
Co-authored-by: turenlong <rentu@microsoft.com>
Co-authored-by: turenlong <rentu@microsoft.com>
* perf: add test files * perf: update test
* test: add browser test * perf: remove useless code
Co-authored-by: turenlong <rentu@microsoft.com>
* ci: add ci for spec-parser * test: update test * test: update test * test: update dependency
* ci: add cd for spec parser * fix: fix pacakge * test: fix ut
* perf: try to fix browser lib issue * perf: add missing constant export --------- Co-authored-by: turenlong <rentu@microsoft.com>
* perf: add format for spec parser * fix: add eslint file * build: update lock file
* perf: add license and update package.json * ci: update version
Co-authored-by: turenlong <rentu@microsoft.com>
if (!newPaths[path]) { | ||
newPaths[path] = { ...unResolveSpec.paths[path] }; | ||
for (const m of ConstantString.AllOperationMethods) { | ||
delete (newPaths[path] as any)[m]; |
Check warning
Code scanning / CodeQL
Prototype-polluting assignment Medium
library input
} | ||
} | ||
|
||
(newPaths[path] as any)[methodName] = (unResolveSpec.paths[path] as any)[methodName]; |
Check warning
Code scanning / CodeQL
Prototype-polluting assignment Medium
library input
config.set(karmaConfig); | ||
}; | ||
|
||
function getTestFiles(argv) { |
Check notice
Code scanning / CodeQL
Unused variable, import, function or class Note
packages/spec-parser/package.json
Outdated
"stream-http": "^3.2.0", | ||
"swagger2openapi": "^7.0.8", | ||
"url": "^0.11.3", | ||
"webpack": "^5.90.0" |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
webpack is a deps? not devDeps?
Codecov ReportAttention:
Additional details and impacted files@@ Coverage Diff @@
## dev #10842 +/- ##
==========================================
+ Coverage 86.88% 87.17% +0.29%
==========================================
Files 451 461 +10
Lines 26644 27335 +691
Branches 5332 5513 +181
==========================================
+ Hits 23149 23829 +680
+ Misses 1768 1759 -9
- Partials 1727 1747 +20
|
No description provided.