Skip to content

Commit f28b9ac

Browse files
authored
Merge pull request #2 from lsst-sqre/u/rra/dependencies
Remove docs and linkcheck builds
2 parents b126171 + 337fc17 commit f28b9ac

2 files changed

Lines changed: 0 additions & 109 deletions

File tree

.github/workflows/ci.yaml

Lines changed: 0 additions & 74 deletions
Original file line numberDiff line numberDiff line change
@@ -62,80 +62,6 @@ jobs:
6262
- "src/pclient/**"
6363
docs-specific:
6464
- "docs/**"
65-
66-
docs:
67-
runs-on: ubuntu-latest
68-
timeout-minutes: 15
69-
70-
needs: changes
71-
if: >-
72-
(needs.changes.outputs.docs == 'true')
73-
|| (github.event_name == 'workflow_dispatch')
74-
|| (github.event_name == 'release' && github.event.action == 'published')
75-
76-
steps:
77-
- uses: actions/checkout@v6
78-
with:
79-
fetch-depth: 0
80-
81-
- name: Install uv
82-
uses: astral-sh/setup-uv@v7
83-
with:
84-
version: ${{ env.UV_VERSION }}
85-
86-
- name: Install extra packages
87-
run: |
88-
sudo apt-get update
89-
sudo apt-get install -y graphviz
90-
91-
- name: Run tox
92-
run: uv run --only-group=tox tox run -e docs
93-
94-
# Upload docs:
95-
# - on pushes to main if *any* documentation content might have changed
96-
# - on workflow dispatches
97-
# - on pushes to tickets/ branches if docs/ directory content changed
98-
- name: Upload to LSST the Docs
99-
uses: lsst-sqre/ltd-upload@v1
100-
with:
101-
project: "pclient"
102-
dir: "docs/_build/html"
103-
username: ${{ secrets.LTD_USERNAME }}
104-
password: ${{ secrets.LTD_PASSWORD }}
105-
if: >-
106-
(github.event_name == 'push' && github.ref_name == 'main')
107-
|| (github.event_name == 'merge_group')
108-
|| (github.event_name == 'workflow_dispatch')
109-
|| (github.event_name == 'pull_request'
110-
&& (startsWith(github.head_ref, 'tickets/')
111-
|| startsWith(github.head_ref, 't/'))
112-
&& steps.filter.outputs.docs-specific == 'true')
113-
114-
linkcheck:
115-
runs-on: ubuntu-latest
116-
timeout-minutes: 15
117-
118-
needs: changes
119-
if: ${{ needs.changes.outputs.docs == 'true' }}
120-
121-
steps:
122-
- uses: actions/checkout@v6
123-
with:
124-
fetch-depth: 0
125-
126-
- name: Install uv
127-
uses: astral-sh/setup-uv@v7
128-
with:
129-
version: ${{ env.UV_VERSION }}
130-
131-
- name: Install extra packages
132-
run: |
133-
sudo apt-get update
134-
sudo apt-get install -y graphviz
135-
136-
- name: Run tox
137-
run: uv run --only-group=tox tox run -e docs-linkcheck
138-
13965
build:
14066
needs: [test]
14167
secrets: inherit

.github/workflows/periodic-ci.yaml

Lines changed: 0 additions & 35 deletions
Original file line numberDiff line numberDiff line change
@@ -44,38 +44,3 @@ jobs:
4444
notification_title: "Periodic test for {repo} failed"
4545
env:
4646
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_ALERT_WEBHOOK }}
47-
48-
docs:
49-
runs-on: ubuntu-latest
50-
timeout-minutes: 10
51-
52-
steps:
53-
- uses: actions/checkout@v6
54-
with:
55-
fetch-depth: 0
56-
57-
- name: Install uv
58-
uses: astral-sh/setup-uv@v7
59-
with:
60-
version: ${{ env.UV_VERSION }}
61-
62-
- name: Install Graphviz
63-
run: |
64-
sudo apt-get update
65-
sudo apt-get install -y graphviz
66-
67-
- name: Update dependencies
68-
run: make update-deps
69-
70-
- name: Run tox
71-
run: uv run --only-group=tox tox run -e docs,docs-linkcheck
72-
73-
- name: Report status
74-
if: failure()
75-
uses: ravsamhq/notify-slack-action@v2
76-
with:
77-
status: ${{ job.status }}
78-
notify_when: "failure"
79-
notification_title: "Periodic documentation test for {repo} failed"
80-
env:
81-
SLACK_WEBHOOK_URL: ${{ secrets.SLACK_ALERT_WEBHOOK }}

0 commit comments

Comments
 (0)