Skip to content

Propagate tomlfmt check failures - #70

Merged
legokichi merged 2 commits into
mainfrom
codex/fmt-check-tomlfmt
Jun 11, 2026
Merged

Propagate tomlfmt check failures#70
legokichi merged 2 commits into
mainfrom
codex/fmt-check-tomlfmt

Conversation

@legokichi

Copy link
Copy Markdown
Member

Summary

  • replace find -execdir ... \; tomlfmt invocations with shell loops that propagate nonzero exits
  • apply the same tomlfmt check behavior in the GitHub Actions fmt job
  • clean generated Cargo.toml.new files before and after dry-run checks

Validation

  • git diff --check -- Makefile .github/workflows/check.yml
  • exact CI shell block: exits 255 and reports a tomlfmt failure in leptos-browser/Cargo.toml, confirming failure propagation
  • make fmt-check: exits 2 via cargo tomlfmt -d error 255 on the same existing manifest formatting issue

@github-actions

Copy link
Copy Markdown

Terraform Plan Results

Terraform Plan Output

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.

Pusher: @legokichi, Action: pull_request

@github-actions

Copy link
Copy Markdown

Terraform Plan Results

Terraform Plan Output

No changes. Your infrastructure matches the configuration.

Terraform has compared your real infrastructure against your configuration
and found no differences, so no changes are needed.

Pusher: @legokichi, Action: pull_request

@legokichi
legokichi marked this pull request as ready for review June 11, 2026 17:04
@legokichi
legokichi merged commit 8988f07 into main Jun 11, 2026
7 checks passed
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