Skip to content

[TypeSpec Validation] Quote argument to "tsp format"#26210

Merged
mikeharder merged 1 commit intoAzure:mainfrom
mikeharder:tsp-format-quote
Oct 13, 2023
Merged

[TypeSpec Validation] Quote argument to "tsp format"#26210
mikeharder merged 1 commit intoAzure:mainfrom
mikeharder:tsp-format-quote

Conversation

@mikeharder
Copy link
Member

@mikeharder mikeharder commented Oct 13, 2023

- Prevents shell expansion which may miss files
@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 13, 2023

Next Steps to Merge

✔️ All automated merging requirements have been met! Refer to step 4 in the PR workflow diagram (even if your PR is for data plane, not ARM).

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 13, 2023

Swagger Validation Report

️️✔️BreakingChange succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️Breaking Change(Cross-Version) succeeded [Detail] [Expand]
There are no breaking changes.
️️✔️CredScan succeeded [Detail] [Expand]
There is no credential detected.
️️✔️LintDiff succeeded [Detail] [Expand]
Validation passes for LintDiff.
️️✔️Avocado succeeded [Detail] [Expand]
Validation passes for Avocado.
️️✔️SwaggerAPIView succeeded [Detail] [Expand]
️️✔️TypeSpecAPIView succeeded [Detail] [Expand]
️️✔️ModelValidation succeeded [Detail] [Expand]
Validation passes for ModelValidation.
️️✔️SemanticValidation succeeded [Detail] [Expand]
Validation passes for SemanticValidation.
️️✔️PoliCheck succeeded [Detail] [Expand]
Validation passed for PoliCheck.
️️✔️PrettierCheck succeeded [Detail] [Expand]
Validation passes for PrettierCheck.
️️✔️SpellCheck succeeded [Detail] [Expand]
Validation passes for SpellCheck.
️️✔️Lint(RPaaS) succeeded [Detail] [Expand]
Validation passes for Lint(RPaaS).
️️✔️PR Summary succeeded [Detail] [Expand]
Validation passes for Summary.
️️✔️Automated merging requirements met succeeded [Detail] [Expand]
Posted by Swagger Pipeline | How to fix these errors?

@openapi-pipeline-app
Copy link

openapi-pipeline-app bot commented Oct 13, 2023

Swagger pipeline restarted successfully, please wait for status update in this comment.

@openapi-pipeline-app
Copy link

Swagger pipeline started successfully. If there is ApiView generated, it will be updated in this comment.

@mikeharder mikeharder assigned mikeharder and unassigned weshaggard Oct 13, 2023
@mikeharder mikeharder requested a review from ckairen October 13, 2023 06:23
@mikeharder mikeharder merged commit 123c2e5 into Azure:main Oct 13, 2023
@openapi-pipeline-app
Copy link

Swagger pipeline restarted successfully, please wait for status update in this comment.

@mikeharder mikeharder deleted the tsp-format-quote branch October 13, 2023 18:51
jnlycklama pushed a commit that referenced this pull request Nov 8, 2023
- Prevents shell expansion which may miss files
- Fixes microsoft/typespec#2568
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Archived in project

Development

Successfully merging this pull request may close these issues.

"tsp format" has different indentation on Linux vs Windows

3 participants