Skip to content

fix: unquoted $@, missing popd, oauth typo, .gitignore filters, and other typos - #224

Merged
k-rister merged 5 commits into
mainfrom
fix-crucible-ci-bugs
Jul 7, 2026
Merged

fix: unquoted $@, missing popd, oauth typo, .gitignore filters, and other typos#224
k-rister merged 5 commits into
mainfrom
fix-crucible-ci-bugs

Conversation

@k-rister

@k-rister k-rister commented Jul 6, 2026

Copy link
Copy Markdown
Contributor

Summary

  • Quote $@ in do_ssh to prevent word splitting
  • Add missing popd and closing quote in check-controller-build
  • Fix "oath" → "oauth" in quay token step names (3 workflow files)
  • Add .gitignore to docs-only file filter lists (3 workflow files)
  • Fix typos across 8 files: descriptiopn, obtian, repoistory, availble, copmatibility, it's/its, custon, Github/GitHub, there/their

Closes #223

Test plan

  • CI passes
  • bash -n on modified shell scripts

🤖 Generated with Claude Code

k-rister and others added 5 commits July 6, 2026 15:36
Closes: #223

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
The pushd into the CI target directory had no matching popd, and
the error message was missing a closing single quote.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
Changes to .gitignore should not trigger CI runs.

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
descriptiopn, obtian, repoistory, availble, copmatibility, it's/its,
custon, Github/GitHub, there/their

Co-Authored-By: Claude Opus 4.6 (1M context) <noreply@anthropic.com>
@k-rister k-rister self-assigned this Jul 6, 2026
@k-rister
k-rister requested a review from a team July 6, 2026 20:42
@project-crucible-tracking project-crucible-tracking Bot moved this to In Progress in Crucible Tracking Jul 6, 2026
@k-rister
k-rister merged commit 47721e2 into main Jul 7, 2026
1874 of 1895 checks passed
@github-project-automation github-project-automation Bot moved this from In Progress to Done in Crucible Tracking Jul 7, 2026
@k-rister
k-rister deleted the fix-crucible-ci-bugs branch July 7, 2026 20:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

Status: Done

Development

Successfully merging this pull request may close these issues.

fix: descriptiopn typo, unquoted $@ in do_ssh, and workflow issues

2 participants