Skip to content

Commit

Permalink
[CI] split conda recipes and linting to separate files in azure pipel…
Browse files Browse the repository at this point in the history
…ines (#2238)

* move functionality to separate yamls

* add endlines

* Update linting.yml

* Update _forest.py

* Revert "Update _forest.py"

This reverts commit a227d5a.

* Update linting.yml

* Update linting.yml

* Update linting.yml

* Update _forest.py

* Update _forest.py
  • Loading branch information
icfaust authored Jan 7, 2025
1 parent f48dad1 commit e9816c2
Show file tree
Hide file tree
Showing 4 changed files with 83 additions and 41 deletions.
44 changes: 3 additions & 41 deletions .ci/pipeline/ci.yml
Original file line number Diff line number Diff line change
Expand Up @@ -48,20 +48,7 @@ jobs:
pool:
vmImage: 'ubuntu-22.04'
steps:
- script: |
echo "##vso[task.setvariable variable=ISORT_VERSION]$(grep -zoP '(?<=isort\n rev: ).*' .pre-commit-config.yaml)"
echo "##vso[task.setvariable variable=BLACK_VERSION]$(grep -zoP '(?<=black\n rev: ).*' .pre-commit-config.yaml)"
echo "##vso[task.setvariable variable=PYTHON_LINT_VERSION]$(grep -oP -m 1 '(?<=python).*' .pre-commit-config.yaml)"
displayName: 'Collect lint versions'
- task: UsePythonVersion@0
inputs:
versionSpec: $(PYTHON_LINT_VERSION)
addToPath: true
- script: |
python -m pip install --upgrade pip setuptools
pip install isort==${ISORT_VERSION} black[jupyter]==${BLACK_VERSION}
isort --profile black --check . && black --check .
displayName: 'Linting'
- template: linting.yml
- job: LinuxCondaRecipe
dependsOn: Lint
timeoutInMinutes: 120
Expand All @@ -73,19 +60,7 @@ jobs:
pool:
vmImage: 'ubuntu-22.04'
steps:
- script: |
conda config --add channels conda-forge
conda config --set channel_priority strict
conda update -y -q --all
displayName: "Conda update"
- script: |
conda create -y -q -n build-env conda-build conda-verify
displayName: "Conda create"
- script: |
. /usr/share/miniconda/etc/profile.d/conda.sh
conda activate build-env
conda build . --python $(PYTHON_VERSION) --numpy $(NUMPY_VERSION)
displayName: "Conda build and test"
- template: conda-recipe-lnx.yml
- job: WindowsCondaRecipe
dependsOn: Lint
timeoutInMinutes: 120
Expand All @@ -97,20 +72,7 @@ jobs:
pool:
vmImage: 'windows-2022'
steps:
- powershell: Write-Host "##vso[task.prependpath]$env:CONDA\Scripts"
displayName: Add conda to PATH
- script: |
conda config --add channels conda-forge
conda config --set channel_priority strict
conda update -y -q --all
displayName: "Conda update"
- script: |
conda create -y -q -n build-env conda-build conda-verify
displayName: "Conda create"
- script: |
call activate build-env
conda build . --python $(PYTHON_VERSION) --numpy $(NUMPY_VERSION)
displayName: "Conda build and test"
- template: conda-recipe-win.yml
- job: LinuxCondaEnv
dependsOn: Lint
timeoutInMinutes: 120
Expand Down
29 changes: 29 additions & 0 deletions .ci/pipeline/conda-recipe-lnx.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,29 @@
#===============================================================================
# Copyright contributors to the oneDAL project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#===============================================================================
steps:
- script: |
conda config --add channels conda-forge
conda config --set channel_priority strict
conda update -y -q --all
displayName: "Conda update"
- script: |
conda create -y -q -n build-env conda-build conda-verify
displayName: "Conda create"
- script: |
. /usr/share/miniconda/etc/profile.d/conda.sh
conda activate build-env
conda build . --python $(PYTHON_VERSION) --numpy $(NUMPY_VERSION)
displayName: "Conda build and test"
30 changes: 30 additions & 0 deletions .ci/pipeline/conda-recipe-win.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,30 @@
#===============================================================================
# Copyright contributors to the oneDAL project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#===============================================================================
steps:
- powershell: Write-Host "##vso[task.prependpath]$env:CONDA\Scripts"
displayName: Add conda to PATH
- script: |
conda config --add channels conda-forge
conda config --set channel_priority strict
conda update -y -q --all
displayName: "Conda update"
- script: |
conda create -y -q -n build-env conda-build conda-verify
displayName: "Conda create"
- script: |
call activate build-env
conda build . --python $(PYTHON_VERSION) --numpy $(NUMPY_VERSION)
displayName: "Conda build and test"
21 changes: 21 additions & 0 deletions .ci/pipeline/linting.yml
Original file line number Diff line number Diff line change
@@ -0,0 +1,21 @@
#===============================================================================
# Copyright contributors to the oneDAL project
#
# Licensed under the Apache License, Version 2.0 (the "License");
# you may not use this file except in compliance with the License.
# You may obtain a copy of the License at
#
# http://www.apache.org/licenses/LICENSE-2.0
#
# Unless required by applicable law or agreed to in writing, software
# distributed under the License is distributed on an "AS IS" BASIS,
# WITHOUT WARRANTIES OR CONDITIONS OF ANY KIND, either express or implied.
# See the License for the specific language governing permissions and
# limitations under the License.
#===============================================================================
steps:
- script: |
python -m pip install --upgrade pip pre-commit==4.0.1
pre-commit install
pre-commit run --all-files --show-diff-on-failure
displayName: 'Linting'

0 comments on commit e9816c2

Please sign in to comment.