Skip to content

Commit 2ca95fb

Browse files
EdwardAngertclaude
andcommitted
fix: inline environment setup instead of using composite action
Replace the problematic docs-setup action with inline setup steps: 1. Move Node.js, PNPM, and Vale setup directly into workflows 2. Use standard GitHub-provided actions for environment setup 3. Update both docs-unified and docs-reusable-example workflows 4. Remove mention of the setup action from docs README This approach avoids the persistent issue with the composite action not being found during workflow execution. 🤖 Generated with [Claude Code](https://claude.ai/code) Co-Authored-By: Claude <noreply@anthropic.com>
1 parent 12ac6f8 commit 2ca95fb

File tree

3 files changed

+43
-8
lines changed

3 files changed

+43
-8
lines changed

.github/docs/README.md

Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -4,7 +4,6 @@ This directory contains GitHub Actions, configurations, and workflows for Coder'
44

55
## Directory Structure
66

7-
- `actions/docs-setup`: Common setup action for documentation workflows
87
- `actions/docs-shared`: Phase-based composite action providing core documentation functionality
98
- `vale`: Configuration and style rules for Vale documentation linting
109
- `.lycheeignore`: Configuration patterns for lychee link checking

.github/workflows/docs-reusable-example.yaml

Lines changed: 21 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -42,11 +42,28 @@ jobs:
4242
egress-policy: audit
4343

4444
# === PHASE 1: SETUP AND CHECKOUT ===
45-
# Use the reusable setup action for efficient environment setup
46-
- name: Setup Documentation Environment
47-
uses: ./.github/docs/actions/docs-setup
45+
- name: Checkout code
46+
uses: actions/checkout@v4
47+
48+
- name: Setup Node.js
49+
uses: actions/setup-node@v4
50+
with:
51+
node-version: '20'
52+
cache: 'pnpm'
53+
54+
- name: Install PNPM
55+
uses: pnpm/action-setup@v3
56+
with:
57+
run_install: false
58+
59+
- name: Install dependencies
60+
run: ./scripts/pnpm_install.sh
61+
62+
- name: Install Vale
63+
uses: errata-ai/vale-action@v2
4864
with:
49-
setup-vale: true
65+
config: .github/docs/vale/.vale.ini
66+
files: "" # Don't run Vale yet, just install it
5067

5168
# === PHASE 2: PR INFORMATION ===
5269
- name: Get PR Information

.github/workflows/docs-unified.yaml

Lines changed: 22 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -57,10 +57,29 @@ jobs:
5757
egress-policy: audit
5858

5959
# Phase 1: Setup and Initial Checkout
60-
- name: Setup Environment
61-
uses: ./.github/docs/actions/docs-setup
60+
- name: Checkout code
61+
uses: actions/checkout@v4
62+
63+
- name: Setup Node.js
64+
uses: actions/setup-node@v4
65+
with:
66+
node-version: '20'
67+
cache: 'pnpm'
68+
69+
- name: Install PNPM
70+
uses: pnpm/action-setup@v3
71+
with:
72+
run_install: false
73+
74+
- name: Install dependencies
75+
run: ./scripts/pnpm_install.sh
76+
77+
- name: Install Vale (if needed)
78+
if: inputs.lint-vale == 'true'
79+
uses: errata-ai/vale-action@v2
6280
with:
63-
setup-vale: ${{ inputs.lint-vale }}
81+
config: .github/docs/vale/.vale.ini
82+
files: "" # Don't run Vale yet, just install it
6483

6584
# Phase 2: Prepare PR Information
6685
- name: Get PR info

0 commit comments

Comments
 (0)