From 5e7986762a6e58f28db88a972879d3e5f2a8acfe Mon Sep 17 00:00:00 2001 From: Brendan Walsh Date: Sat, 4 Apr 2026 03:21:06 +0000 Subject: [PATCH 1/2] ci: remove unnecessary conda env and Azure auth from Style job MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The Style job was using AzureCLI@2 for sbt scalastyle (which doesn't need Azure auth — Secrets.scala values are all lazy) and installing the full conda environment (R, torch, horovod, langchain, etc.) just to run black. Replace with direct sbt invocation and pip install of black[jupyter]. Validated in a clean container: both scalastyle and black produce identical results without conda or Azure auth. History: Python style was originally added via pip install (PR #1520, Jun 2022). The conda template was added later during a refactor, not for any functional reason. --- pipeline.yaml | 10 ++-------- 1 file changed, 2 insertions(+), 8 deletions(-) diff --git a/pipeline.yaml b/pipeline.yaml index e9af40f543..00676d58c1 100644 --- a/pipeline.yaml +++ b/pipeline.yaml @@ -101,17 +101,11 @@ jobs: pool: vmImage: $(UBUNTU_VERSION) steps: - - task: AzureCLI@2 + - bash: sbt scalastyle test:scalastyle displayName: 'Scala Style Check' - inputs: - azureSubscription: 'SynapseML Build' - scriptLocation: inlineScript - scriptType: bash - inlineScript: 'sbt scalastyle test:scalastyle' - - template: templates/conda.yml - bash: | set -e - source activate synapseml + pip install -q "black[jupyter]==22.3.0" black --diff --color . && black --check -q . displayName: 'Python Style Check' - ${{ if eq(parameters.publishArtifacts, true) }}: From 694f769249d999580ea46d00086199468fb222cd Mon Sep 17 00:00:00 2001 From: Brendan Walsh Date: Sat, 4 Apr 2026 03:27:39 +0000 Subject: [PATCH 2/2] ci: pin Python version for style check MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit Address review feedback — add UsePythonVersion to ensure a consistent Python runtime on the hosted agent. --- pipeline.yaml | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/pipeline.yaml b/pipeline.yaml index 00676d58c1..589b9d72c5 100644 --- a/pipeline.yaml +++ b/pipeline.yaml @@ -103,6 +103,10 @@ jobs: steps: - bash: sbt scalastyle test:scalastyle displayName: 'Scala Style Check' + - task: UsePythonVersion@0 + displayName: 'Use Python 3' + inputs: + versionSpec: '3.10' - bash: | set -e pip install -q "black[jupyter]==22.3.0"