-
Notifications
You must be signed in to change notification settings - Fork 245
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
VSCode extension: add openapi3 preview #6015
Open
archerzz
wants to merge
72
commits into
microsoft:main
Choose a base branch
from
archerzz:vscode/openapi-preview
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
+2,001
−1,860
Open
Changes from 1 commit
Commits
Show all changes
72 commits
Select commit
Hold shift + click to select a range
ac13a8e
init check-in
8e9c36b
do not use tmp folder
5f4bba3
open preview panel only after swagger file is generated
64e14d8
Merge branch 'main' into vscode/openapi-preview
63492f3
improve multiple swagger file selection ui
556e9db
use temp folder to host preview output
9166069
Merge branch 'main' into vscode/openapi-preview
f057599
fix pnpm not supprted in vsce package
c202688
fix command title
84dd3f8
Check minimal version of compiler
baae442
Merge branch 'main' into vscode/openapi-preview
e87fc7c
improve file selection error
b5a1f69
improve error handling
e224bde
add chronus change log
9185597
Merge branch 'main' into vscode/openapi-preview
archerzz ae60f8c
fix lint and format errors
fcd4a5c
roll back changes in license notice
68727e3
update change log to remove [vscode]
f639b31
add `swagger-ui-dist` as dependency and update build scripts accordingly
29b71e5
replace showErrorMessage with logger.error
0388c2b
catch async io operation exceptions
a3ffb90
roll back the change of throwing exception from `runCliCommand`
8014ae4
move `throttle()` from command implementation into `utils.ts`
c7b3238
move `parseOpenApi3File()` into `utils.ts`
3c87bd9
replace methods of `path` module with `path-utils` from compiler
3c5e28c
move `openaapi3-preview.ts` into `vscode-cmd` folder
71fdb8f
add openapi3 preview into menu items
ea6f6f7
update `showQuickPick` options to follow the convention
5d23d15
prompt uses to select when multiple output files are available
4a11104
Merge branch 'main' into vscode/openapi-preview
872f302
update pnpm-lock
ff30c0e
restore license notice
bcdcb52
Merge branch 'main' into vscode/openapi-preview
archerzz 76d9c8f
http-client-java, when codegen error, trace the stderr too (#6054)
weidongxu-microsoft ac0b22d
Add details for C# emitter (#6052)
JoshLove-msft 2ec8fae
[@typespec/http-specs] Fixed paths in specs/routes (#6013)
v-jiaodi 2571810
Respect fully qualified namespace when matching (#6051)
JoshLove-msft d3d8482
refactor: rename ClientModelPlugin to ScmCodeModelPlugin (#6055)
jorgerangel-msft 53cc8d8
Add additional options to readme (#6057)
JoshLove-msft 266342e
Fix: StringTemplate type not supported in typespecValueToJson (#5937)
timotheeguerin 8b397be
Add tests for includeTemplateDeclaration (#5894)
timotheeguerin df4f95b
Add fully qualified names support for AddTypesToKeep (#6061)
JoshLove-msft 59ff8bc
Delete out-dated doc (#6063)
JoshLove-msft 72492cd
Standalone ability to start local server (#6047)
timotheeguerin d67eb8c
remove typespec extension out of preview model (#5900)
chunyu3 1030b4a
[python] remove confusing words from docstring of models (#6070)
msyyc 56316b9
Avoid duplicate null check and fix null serialization property name (…
JoshLove-msft 7174976
Fixes to enums, operation signatures (#5952)
markcowl 10883fc
Build emitter tests (#6089)
JoshLove-msft 69bf3ce
update ordering of commands and menu items
ab1889e
rename `parseOpenApi3File` to `parseJsonFromFile` and change error ha…
ef3981c
add selected file in the error log if it's available
ed00127
Merge branch 'main' into vscode/openapi-preview
archerzz 3bce4db
Merge branch 'main' into vscode/openapi-preview
archerzz 8f74429
Merge branch 'main' into vscode/openapi-preview
archerzz f403f70
Merge branch 'main' into vscode/openapi-preview
archerzz 0c85f2f
re-use `TraverseMainTspFileInWorkspace()` in `typespec-utils.ts`
c3e39de
do not show pop-up message when user cancels file selection
5d520df
move `swagger-ui-dist` from `dependencies` to `devDependencies`
48dbbe1
use rollup to copy swaigger-ui-dist files
ee3a135
Merge branch 'main' into vscode/openapi-preview
archerzz 331537d
Merge branch 'main' into vscode/openapi-preview
d167658
remove `--no-dependencies` in `vsce package` command
59c0cc2
replace custom temp folder creation codes with `mkdtemp()`
88607c7
Merge branch 'main' into vscode/openapi-preview
0b15a08
clear tmp folder before generate open api3 contents
5907562
fix overwritten package dependencies
3caedbf
Merge branch 'main' into vscode/openapi-preview
e1b76ae
parse `packge.json` for entry point tsp file
f8d3765
unify swagger-ui-dist versions
5e4c272
Merge branch 'main' into vscode/openapi-preview
fe2ea06
Merge branch 'main' into vscode/openapi-preview
archerzz File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
move
swagger-ui-dist
from dependencies
to devDependencies
- Loading branch information
There are no files selected for viewing
This file contains bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Some generated files are not rendered by default. Learn more about how customized files appear on GitHub.
Oops, something went wrong.
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
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.
seems to be some version downgrade happening here, probably a merge issue.
can you run this (replace upstream with the name of the upstream remote(this repo) if its not that)