Skip to content

chore: format all Dart and Python files#5773

Merged
beastoin merged 2 commits intomainfrom
chore/format-all-files
Mar 18, 2026
Merged

chore: format all Dart and Python files#5773
beastoin merged 2 commits intomainfrom
chore/format-all-files

Conversation

@beastoin
Copy link
Copy Markdown
Collaborator

Summary

  • Run dart format --line-length 120 across all 456 Dart files in app/
  • Run black --line-length 120 --skip-string-normalization across 10 Python files in backend/
  • ARB files checked — all 33 non-English locales already use correct 4-space indent
  • C/C++ firmware files skipped (clang-format not available on CI runner)

Formatting rules match the pre-commit hook in scripts/pre-commit.

Test plan

  • Verify no logic changes — formatting only
  • app/test.sh passes
  • backend/test.sh passes

🤖 Generated with Claude Code

beastoin and others added 2 commits March 18, 2026 04:45
Run dart format across all 456 Dart files to fix formatting drift.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
Run black formatter across all 10 Python files that had formatting drift.

Co-Authored-By: Claude Opus 4.6 <noreply@anthropic.com>
@beastoin beastoin merged commit 5615252 into main Mar 18, 2026
1 check passed
@beastoin beastoin deleted the chore/format-all-files branch March 18, 2026 04:46
Glucksberg pushed a commit to Glucksberg/omi-local that referenced this pull request Apr 28, 2026
## Summary
- Run `dart format --line-length 120` across all 456 Dart files in
`app/`
- Run `black --line-length 120 --skip-string-normalization` across 10
Python files in `backend/`
- ARB files checked — all 33 non-English locales already use correct
4-space indent
- C/C++ firmware files skipped (clang-format not available on CI runner)

Formatting rules match the pre-commit hook in `scripts/pre-commit`.

## Test plan
- [ ] Verify no logic changes — formatting only
- [ ] `app/test.sh` passes
- [ ] `backend/test.sh` passes

🤖 Generated with [Claude Code](https://claude.com/claude-code)
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