Skip to content

Commit

Permalink
Stop excluding MTRBaseClusters files from restyling.
Browse files Browse the repository at this point in the history
It looks like the restyle job and the zap job agree on them now, as
long as we avoid the failure case of an input on which running
clang-format once produces different output from running clang-format
twice.

Fixes project-chip#20236
  • Loading branch information
bzbarsky-apple committed Oct 6, 2022
1 parent 40a9f35 commit cbca422
Show file tree
Hide file tree
Showing 3 changed files with 401 additions and 351 deletions.
2 changes: 0 additions & 2 deletions .restyled.yaml
Original file line number Diff line number Diff line change
Expand Up @@ -72,8 +72,6 @@ exclude:
- "scripts/idl/tests/outputs/**/*" # Matches generated output 1:1
- "examples/chef/sample_app_util/test_files/*.yaml"
- "examples/chef/zzz_generated/**/*"
- "src/darwin/Framework/CHIP/zap-generated/MTRBaseClusters.mm" # https://github.com/project-chip/connectedhomeip/issues/20236
- "src/darwin/Framework/CHIP/zap-generated/MTRBaseClusters.h" # https://github.com/project-chip/connectedhomeip/issues/20236
- "examples/platform/nxp/k32w/k32w0/scripts/demo_generated_certs/**/*"


Expand Down
2 changes: 1 addition & 1 deletion src/darwin/Framework/CHIP/templates/MTRBaseClusters.zapt
Original file line number Diff line number Diff line change
Expand Up @@ -30,7 +30,7 @@ NS_ASSUME_NONNULL_BEGIN
*
* {{description}}
*/
{{~#*inline "cluster"}}{{asUpperCamelCase parent.name preserveAcronyms=true}}{{/inline~}}
{{#*inline "cluster"}}{{asUpperCamelCase parent.name preserveAcronyms=true}}{{/inline~}}
{{~#*inline "command"}}{{asUpperCamelCase name preserveAcronyms=true}}{{/inline~}}
- (void){{asLowerCamelCase name}}WithParams:(MTR{{>cluster}}Cluster{{>command}}Params * {{#unless (commandHasRequiredField .)}}_Nullable{{/unless}})params completion:({{>command_completion_type command=.}})completion;
{{#unless (hasArguments)}}
Expand Down
Loading

0 comments on commit cbca422

Please sign in to comment.