Skip to content

[codex] Add SwiftFormat checks and Git hooks - #30

Merged
ygsgdbd merged 3 commits into
mainfrom
codex/add-swiftformat-pr-checks
Jul 13, 2026
Merged

[codex] Add SwiftFormat checks and Git hooks#30
ygsgdbd merged 3 commits into
mainfrom
codex/add-swiftformat-pr-checks

Conversation

@ygsgdbd

@ygsgdbd ygsgdbd commented Jul 13, 2026

Copy link
Copy Markdown
Owner

Summary

  • pin SwiftFormat 0.62.1 with a conservative repository configuration and checksum-verified CI download
  • add just commands for setup, formatting, linting, testing, and combined checks
  • add a repository-managed pre-commit hook that formats fully staged Swift files and safely rejects partially staged files
  • add parallel PR checks for formatting and the complete XCTest suite
  • apply the initial mechanical SwiftFormat baseline and document setup in English and Chinese

Developer workflow

brew install just swiftformat tuist
just setup
just check

Local commands only validate installed tools and configure the hook; they do not install or upgrade software.

Validation

  • just lint: 0/28 files require formatting, 7 generated paths skipped
  • just test: 140 tests, 0 failures
  • actionlint .github/workflows/pr-checks.yml
  • pre-commit verification for no Swift changes, formatted staged files, unformatted staged files, and partially staged files
  • SwiftFormat missing, older, newer, and exact-version checks
  • git diff --check origin/main...HEAD

@ygsgdbd
ygsgdbd marked this pull request as ready for review July 13, 2026 09:40
@chatgpt-codex-connector

Copy link
Copy Markdown

You have reached your Codex usage limits for code reviews. You can see your limits in the Codex usage dashboard.

@ygsgdbd
ygsgdbd merged commit 514eb07 into main Jul 13, 2026
2 checks passed
@ygsgdbd
ygsgdbd deleted the codex/add-swiftformat-pr-checks branch July 13, 2026 09:41
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