Skip to content

Add source formatting using clang-format#4

Open
dekrain wants to merge 7 commits into
jabacat:mainfrom
dekrain:style
Open

Add source formatting using clang-format#4
dekrain wants to merge 7 commits into
jabacat:mainfrom
dekrain:style

Conversation

@dekrain

@dekrain dekrain commented Dec 30, 2022

Copy link
Copy Markdown
Collaborator

More suggestions are welcome for debate.

@JakeRoggenbuck

Copy link
Copy Markdown
Member

imo this is good, but still waiting on others to approve.

Comment thread scripts/reformat-all.sh Outdated
@dekrain

dekrain commented Dec 30, 2022

Copy link
Copy Markdown
Collaborator Author

Now I don't think there should be issues with finding the files, unless some of them contain whitespace (thank default unix shell behavior for that).

@adamhutchings adamhutchings left a comment

Copy link
Copy Markdown
Contributor

Choose a reason for hiding this comment

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

LGTM!

@dekrain

dekrain commented Jan 3, 2023

Copy link
Copy Markdown
Collaborator Author

I just pushed a merge commit, and it's not shown here for some reason.

@dekrain dekrain closed this by deleting the head repository Jan 3, 2023
@dekrain dekrain reopened this Jan 3, 2023
@dekrain dekrain requested a review from adamhutchings January 3, 2023 23:55
@adamhutchings adamhutchings mentioned this pull request Jan 4, 2023
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.

3 participants