Skip to content

chore: add .clang-format for C++/ObjC code formatting#7340

Merged
janicduplessis merged 1 commit intodevelopfrom
@janic/add-clang-format
Apr 13, 2026
Merged

chore: add .clang-format for C++/ObjC code formatting#7340
janicduplessis merged 1 commit intodevelopfrom
@janic/add-clang-format

Conversation

@janicduplessis
Copy link
Copy Markdown
Contributor

@janicduplessis janicduplessis commented Apr 10, 2026

Fixes APP-3625

What changed (plus any additional context for devs)

Extracted from the new arch migration PR #6924.

Adds a .clang-format config for consistent formatting of native C++ and Objective-C code. New arch requires writing native module code (Fabric components, TurboModules) in C++/ObjC, so having a formatter config ensures consistent style.

Currently not enforced, can consider that later, but tooling will pick it up.

What to test

  • No runtime impact, config file only

Adds clang-format config for consistent formatting of native code
(C++ and Objective-C). Needed for new arch native module code.
@linear
Copy link
Copy Markdown

linear Bot commented Apr 10, 2026

@janicduplessis janicduplessis requested a review from olerass April 10, 2026 20:19
@janicduplessis janicduplessis enabled auto-merge (squash) April 10, 2026 20:20
@github-actions
Copy link
Copy Markdown

🧪 Flashlight Performance Report (AWS Device Farm)

🔀 Commit: 49dbedd

📎 View Artifacts

Metric Current Δ vs Baseline
Time to Interactive (TTI) 5515 ms
Average FPS 57.03
Average RAM 395.5 MB

@github-actions
Copy link
Copy Markdown

Launch in simulator or device for 49dbedd

Copy link
Copy Markdown
Contributor

@olerass olerass left a comment

Choose a reason for hiding this comment

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

LGTM. Seems to come from react-native defaults -- could mention this in the PR description since it's not currently clear if this was just some custom config we came up with.

@janicduplessis janicduplessis merged commit 291f2d4 into develop Apr 13, 2026
20 checks passed
@janicduplessis janicduplessis deleted the @janic/add-clang-format branch April 13, 2026 08:06
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.

2 participants