Skip to content

ci: run bootstrap-from-hub bash tests on PR - #8

Closed
mrhegemon wants to merge 2 commits into
feat/bootstrap-from-hubfrom
feat/bootstrap-ci
Closed

mrhegemon wants to merge 2 commits into
feat/bootstrap-from-hubfrom
feat/bootstrap-ci

Conversation

@mrhegemon

Copy link
Copy Markdown
Member

Wires the 13 bash tests from #7 into GitHub Actions so script edits don't silently regress.

  • Triggers on PRs touching `bootstrap-from-hub.sh`, `entrypoint.sh`, `tests/`, or the workflow itself.
  • ~5 second runtime, no Docker required.

🤖 Generated with Claude Code

Wires the 13 bash tests added in the bootstrap PR into GitHub Actions
so script edits don't silently regress.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>

@chatgpt-codex-connector chatgpt-codex-connector Bot 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.

💡 Codex Review

Here are some automated review suggestions for this pull request.

Reviewed commit: ca11369685

ℹ️ About Codex in GitHub

Your team has set up Codex to review pull requests in this repo. Reviews are triggered when you

  • Open a pull request for review
  • Mark a draft as ready
  • Comment "@codex review".

If Codex has suggestions, it will comment; otherwise it will react with 👍.

Codex can also answer questions or update the PR. Try commenting "@codex address that feedback".

Comment thread .github/workflows/bootstrap-test.yml Outdated
- .github/workflows/bootstrap-test.yml
push:
branches:
- master

Copy link
Copy Markdown

Choose a reason for hiding this comment

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

P2 Badge Run push CI on the actual default branch

This push trigger is limited to master, but the repository branch in this checkout is main, so merging a PR into the main line will not run these bootstrap tests after merge. The PR path filter still covers PRs, but any direct push or merge commit on main skips the workflow; use main here or remove the branch restriction if both branch names are possible.

Useful? React with 👍 / 👎.

Tests are intended to run locally on demand via
`bash tests/bootstrap-from-hub.test.sh`, not as a GH Actions workflow.

Co-Authored-By: Claude Opus 4.7 (1M context) <noreply@anthropic.com>
@mrhegemon

Copy link
Copy Markdown
Member Author

Closing — these tests are intended to run locally on demand, not as a GH Actions workflow. The bash tests themselves already live in the base PR (which is locally runnable as bash tests/bootstrap-from-hub.test.sh). No CI wiring needed.

@mrhegemon mrhegemon closed this May 28, 2026
@mrhegemon
mrhegemon deleted the feat/bootstrap-ci branch June 30, 2026 14:44
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.

1 participant