Skip to content

Commit 039bd5e

Browse files
nturinskiNathan TurinskiCopilot
authored
Replace JsonCli tool with direct nupkg parsing and native dotnet new (#4947)
* Use .NET list and properties to parse C# template. * Remove uneeded shims * Apply suggestion from @Copilot Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com> * PR feedback --------- Co-authored-by: Nathan Turinski <naturins@microsoft.comm> Co-authored-by: Copilot <175728472+Copilot@users.noreply.github.com>
1 parent 710b226 commit 039bd5e

38 files changed

+312
-3075
lines changed

.azure-pipelines/1esmain.yml

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -51,7 +51,7 @@ extends:
5151
credscan:
5252
suppressionsFile: $(Build.SourcesDirectory)/.azure-pipelines/compliance/CredScanSuppressions.json
5353
codeql:
54-
language: javascript # only build a codeql database for javascript, since the jsoncli pipeline handles csharp
54+
language: javascript
5555
# enabled: true # TODO: would like to enable only on scheduled builds but CodeQL cannot currently be disabled per https://eng.ms/docs/cloud-ai-platform/devdiv/one-engineering-system-1es/1es-docs/codeql/1es-codeql
5656
pool:
5757
name: VSEngSS-MicroBuild2022-1ES # Name of your hosted pool

.vscode/settings.json

Lines changed: 0 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -7,9 +7,6 @@
77
"editor.formatOnSave": true,
88
"editor.insertSpaces": true,
99
"editor.tabSize": 4,
10-
"files.exclude": {
11-
"tools/JsonCli/src/**": true
12-
},
1310
"files.insertFinalNewline": true,
1411
"files.trimTrailingWhitespace": true,
1512
"search.exclude": {
-73.4 KB
Binary file not shown.
-28.4 KB
Binary file not shown.
-190 KB
Binary file not shown.
Binary file not shown.
-113 KB
Binary file not shown.
-112 KB
Binary file not shown.

resources/dotnetJsonCli/Microsoft.TemplateEngine.JsonCli.deps.json

Lines changed: 0 additions & 1685 deletions
This file was deleted.
-34.4 KB
Binary file not shown.

0 commit comments

Comments
 (0)