Skip to content

ci: removed multiple unused CI scripts - #7641

Open
knst wants to merge 2 commits into
dashpay:developfrom
knst:cleanup-ci-unused
Open

ci: removed multiple unused CI scripts#7641
knst wants to merge 2 commits into
dashpay:developfrom
knst:cleanup-ci-unused

Conversation

@knst

@knst knst commented Aug 27, 2026

Copy link
Copy Markdown
Collaborator

Issue being fixed or feature implemented

Some scripts for "CI" exists as they have been backported from Bitcoin Core but they are completely irrelevant, in non-working condition.
Beside extra conflicts to resolve for Dash Core when backporting changes from Bitcoin Core, they also masks real changes that should be done during backports but in other files, such as ci/dash/build_src.sh or ci/dash/lint-tidy.sh.

What was done?

Removed:

  • ci/test/04_install.sh
  • ci/test/05_before_script.sh
  • ci/test/wrap-wine.sh
  • ci/test_run_all.sh
  • ci/dash/build_depends.sh

Updated ci/README.md accordingly current status.

How Has This Been Tested?

N/A

Breaking Changes

N/A

Checklist:

  • I have performed a self-review of my own code
  • I have commented my code, particularly in hard-to-understand areas
  • I have added or updated relevant unit/integration/functional/e2e tests
  • I have made corresponding changes to the documentation
  • I have assigned this pull request to a milestone (for repository code-owners and collaborators only)

knst added 2 commits August 27, 2026 23:57
Nothing references ci/dash/build_depends.sh: the depends build moved
into .github/workflows/build-depends.yml, which runs `make -C depends`
inline.
Dash CI runs on GitHub Actions via ci/dash/*.sh, which selects a ci/test/00_setup_env*.sh configuration through ci/dash/matrix.sh.
The staged Bitcoin Core pipeline was never wired into it: ci/test_run_all.sh sources 04_install.sh and 05_before_script.sh and then stops, because the 06_script_*.sh stages were never adopted, so it sets up an environment without running a single test.
Nothing references these files except ci/README.md, yet they keep absorbing backport conflicts and maintenance that has no effect on CI.

Same reasoning as the earlier .cirrus.yml (dashpay#6704) removal

Also removed as fully unreferenced: ci/test/wrap-wine.sh

ci/README.md is trimmed accordingly; its removed examples also cited
ci/test/00_setup_env_arm.sh, a file that does not exist.
@knst knst added this to the 24 milestone Aug 27, 2026
@thepastaclaw

thepastaclaw commented Aug 27, 2026

Copy link
Copy Markdown

✅ Final review complete — no blockers (commit e9b40c4)

@coderabbitai

coderabbitai Bot commented Aug 27, 2026

Copy link
Copy Markdown

Review Change Stack

No actionable comments were generated in the recent review. 🎉

ℹ️ Recent review info
⚙️ Run configuration

Configuration used: Repository UI

Review profile: CHILL

Plan: Pro Plus

Run ID: 286a83e3-750f-4236-860e-f1388a4c04e9

📥 Commits

Reviewing files that changed from the base of the PR and between d1f7ff5 and e9b40c4.

📒 Files selected for processing (6)
  • ci/README.md
  • ci/dash/build_depends.sh
  • ci/test/04_install.sh
  • ci/test/05_before_script.sh
  • ci/test/wrap-wine.sh
  • ci/test_run_all.sh
💤 Files with no reviewable changes (5)
  • ci/test/wrap-wine.sh
  • ci/test/05_before_script.sh
  • ci/test_run_all.sh
  • ci/dash/build_depends.sh
  • ci/test/04_install.sh

Included review availability: Your plan provides up to 4 included reviews per hour; 3 remain after this review.


Walkthrough

The CI README now documents configuration files selected by BUILD_TARGET and driven by GitHub Actions workflows. It identifies 00_setup_env.sh as the default configuration with fallback values. The change also removes legacy dependency, installation, pre-script, Wine wrapper, and test orchestration scripts.

Estimated code review effort: 2 (Simple) | ~10 minutes

Merge Risk: ⚪ Minimal · up to e9b40

This change removes unused CI scripts and updates the CI documentation without introducing an actionable merge-blocking risk; it is merge-ready after normal checks and review.

🚥 Pre-merge checks | ✅ 5
✅ Passed checks (5 passed)
Check name Status Explanation
Title check ✅ Passed The title clearly and concisely describes the main change: removal of multiple unused CI scripts.
Description check ✅ Passed The description directly explains why the unused CI scripts were removed, lists the affected files, and notes the documentation update.
Docstring Coverage ✅ Passed No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0…
Linked Issues check ✅ Passed Check skipped because no linked issues were found for this pull request.
Out of Scope Changes check ✅ Passed Check skipped because no linked issues were found for this pull request.
Full details: Docstring Coverage

Explanation

No functions found in the changed files to evaluate docstring coverage. Skipping docstring coverage check. Docstring coverage is scoped to functions touched by this diff. Analyzed 0 functions across 0 files. (1 skipped: 1 unsupported.)

✨ Finishing Touches
🧪 Generate unit tests (beta)
  • Create PR with unit tests

Thanks for using CodeRabbit! It's free for OSS, and your support helps us grow. If you like it, consider giving us a shout-out.

❤️ Share

Comment @coderabbitai help to get the list of available commands.

@thepastaclaw thepastaclaw left a comment

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

Final validation — Codex/Sol only (Phase 2 disabled)

At exact head e9b40c4, the PR cleanly removes the obsolete staged CI pipeline and superseded depends helper while updating ci/README.md to describe the active GitHub Actions setup. The active workflows source ci/dash/matrix.sh, build depends directly, and retain no references to the deleted entry points, so no in-scope defect was found.
Source: codex-general, codex-dash-core-commit-history, and codex-backport-reviewer (their exact backend model IDs are not exposed in the supplied evidence); final verifier backend model grok-4.5. Orchestration-only: openclaw-agent/cliproxy/gpt-5.6-sol (not reviewer evidence).

Validated zero-blocker Codex/Sol precheck evidence was promoted to final because Phase 2 (Sonnet/Opus) is temporarily disabled. This is Codex/Sol-only final validation, not Codex + Sonnet/Opus coverage.

Review provenance

  • Codex reviewers: gpt-5.6-sol — general (completed), gpt-5.6-sol — dash-core-commit-history (completed), gpt-5.6-sol — backport-reviewer (completed)
  • Verifier: gpt-5.6-sol — verifier
  • Sonnet/Opus: not run (Phase 2 disabled — temporary Codex/Sol-only final)
  • Secondary pass: disabled (temporary_phase2_sonnet_disable)

@github-actions

github-actions Bot commented Aug 27, 2026

Copy link
Copy Markdown

Potential PR merge conflicts

This is advisory only. It does not block CI, but it marks PRs that will likely need a rebase depending on merge order.

If these PRs merge first

This PR will likely need a rebase:

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants