Skip to content

Commit bc11e79

Browse files
committed
gate lint with linux
1 parent 837a6b4 commit bc11e79

File tree

2 files changed

+18
-12
lines changed

2 files changed

+18
-12
lines changed

ci/steps.common.yml

Lines changed: 13 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -2,8 +2,9 @@ parameters:
22
name: Linux
33

44
steps:
5-
- script: python -m scripts.lint
6-
displayName: Lint
5+
- ${{ if eq(parameters.name, 'Linux') }}:
6+
- script: python -m scripts.lint
7+
displayName: Lint
78

89
- ${{ if not(eq(parameters.name, 'Windows')) }}:
910
- script: python setup.py sdist --dist-dir=_artifacts/sdist
@@ -24,16 +25,6 @@ steps:
2425
- script: BROWSER=headlesschrome python -m scripts.atest
2526
displayName: Test on Chrome
2627

27-
- ${{ if eq(parameters.name, 'Linux') }}:
28-
- script: sphinx-build -M html docs _artifacts/docs
29-
displayName: Build Docs
30-
- task: PublishPipelineArtifact@0
31-
displayName: Publish Docs
32-
inputs:
33-
targetPath: _artifacts/docs/html
34-
artifactName: JupyterLibrary Docs
35-
condition: always()
36-
3728
- task: PublishTestResults@2
3829
displayName: Publish Test Results
3930
inputs:
@@ -46,3 +37,13 @@ steps:
4637
targetPath: _artifacts/test_output
4738
artifactName: JupyterLibrary ${{ parameters.name }} Robot Logs
4839
condition: always()
40+
41+
- ${{ if eq(parameters.name, 'Linux') }}:
42+
- script: sphinx-build -M html docs _artifacts/docs
43+
displayName: Build Docs
44+
- task: PublishPipelineArtifact@0
45+
displayName: Publish Docs
46+
inputs:
47+
targetPath: _artifacts/docs/html
48+
artifactName: JupyterLibrary Docs
49+
condition: always()

scripts/lint.py

Lines changed: 5 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -1,3 +1,8 @@
1+
""" Apply source normalization and checking
2+
3+
This should run without JupyterLibrary installed, but only needs to
4+
work on Python3
5+
"""
16
from pathlib import Path
27
from subprocess import check_call
38

0 commit comments

Comments
 (0)