From 2a2c0b7e033454f405b27ed0dcb657181b7a1205 Mon Sep 17 00:00:00 2001 From: "Collin M. Barrett" Date: Sat, 4 Jul 2020 15:16:23 -0500 Subject: [PATCH] feat(data): lint json changes before migrating --- server/azure-pipelines.migrate.yaml | 39 ++++++++++++++++++++++------- 1 file changed, 30 insertions(+), 9 deletions(-) diff --git a/server/azure-pipelines.migrate.yaml b/server/azure-pipelines.migrate.yaml index 7dbf68435..91f916d34 100644 --- a/server/azure-pipelines.migrate.yaml +++ b/server/azure-pipelines.migrate.yaml @@ -41,17 +41,11 @@ steps: echo "##vso[task.setvariable variable=aborted;isOutput=true]false" fi - - task: UseDotNet@2 - displayName: use latest dotnet sdk + - task: ShellScript@2 + displayName: lint json condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded()) inputs: - version: 3.x - - - task: CmdLine@2 - displayName: install ef dotnet tool - condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded()) - inputs: - script: dotnet tool install -g dotnet-ef + scriptPath: data/lint.sh - task: Bash@3 displayName: git config @@ -65,6 +59,33 @@ steps: git config --global user.name "$GITHUBNAME" git config --global user.email "$GITHUBEMAIL" + - task: Bash@3 + displayName: commit lint results + name: commitLint + condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded()) + inputs: + targetType: inline + workingDirectory: data + script: | + DIFF=$(git status -s -- . | wc -l) + echo "$DIFF file(s) required linting" + if (( $DIFF > 0 )) ; then + git add . + git commit -m "refactor(data): lint PR #$SYSTEM_PULLREQUEST_PULLREQUESTNUMBER" + fi + + - task: UseDotNet@2 + displayName: use latest dotnet sdk + condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded()) + inputs: + version: 3.x + + - task: CmdLine@2 + displayName: install ef dotnet tool + condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded()) + inputs: + script: dotnet tool install -g dotnet-ef + - task: Bash@3 displayName: revert existing PR migration condition: and(eq(variables['abortJustMigrated.aborted'], 'false'), succeeded())