Skip to content

fix: remove validate CI job #6

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Merged
Svtter merged 1 commit into from
Apr 9, 2026
Merged

fix: remove validate CI job #6

Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
26 changes: 0 additions & 26 deletions .github/workflows/ci.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -9,32 +9,6 @@ permissions:
contents: read

jobs:
validate:
runs-on: ubuntu-latest
steps:
- name: Checkout repository
uses: actions/checkout@v6

- name: Install shellcheck
run: |
sudo apt-get update
sudo apt-get install -y shellcheck

- name: Lint shell scripts
run: |
shellcheck setup-opencode/install-opencode.sh
shellcheck github-run-opencode/run-github-opencode.sh
shellcheck run-opencode/run-opencode.sh
shellcheck tests/setup-opencode.sh
shellcheck tests/run-opencode.sh
shellcheck tests/github-run-opencode.sh
shellcheck tests/review-action.sh
shellcheck tests/test.sh
shellcheck tests/fixtures/fake-installer.sh

- name: Run local test suite
run: bash tests/test.sh

smoke-actions:
Copy link
Copy Markdown

@chatgpt-codex-connector chatgpt-codex-connector Bot Apr 9, 2026

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

P1 Badge Restore running the local test suite in CI

This change removes the only CI step that executes bash tests/test.sh, so pull requests now pass with just the smoke-actions happy-path workflow. That smoke job uses single-attempt settings (install-attempts: 1, attempts: 1) and does not exercise many guardrails covered by the local tests (for example, the invalid-attempt validation and retry behavior in tests/setup-opencode.sh and tests/run-opencode.sh), so regressions in the action scripts can be merged undetected.

Useful? React with 👍 / 👎.

runs-on: ubuntu-latest
steps:
Expand Down